Skip to content
This repository has been archived by the owner on Feb 21, 2024. It is now read-only.

feat: allow usage with [email protected] #34

Merged
merged 1 commit into from
Aug 29, 2023
Merged

feat: allow usage with [email protected] #34

merged 1 commit into from
Aug 29, 2023

Conversation

addaleax
Copy link
Contributor

No description provided.

@addaleax addaleax force-pushed the mongodb-6 branch 2 times, most recently from 2e056a8 to 1af2c8a Compare August 29, 2023 10:11
@addaleax addaleax changed the title chore: allow usage with [email protected] feat: allow usage with [email protected] Aug 29, 2023
Comment on lines +44 to 45
"mongodb": "^5.4.0 || ^6.0.0",
"mongodb-log-writer": "^1.2.0"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not know that you could use || in there. TIL.

@@ -134,7 +134,7 @@ async function resolveMongodbSrv(uri: string, logger: ConnectLogEmitter): Promis
resolutionDetails.push({
query: 'SRV', hostname, error: args[0]?.message, wasNativelyLookedUp: wasNativelyLookedUp(args[1])
});
// eslint-disable-next-line node/no-callback-literal
// eslint-disable-next-line n/no-callback-literal
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just out of curiosity: why was this renamed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@addaleax addaleax merged commit 5e8d2ea into main Aug 29, 2023
@addaleax addaleax deleted the mongodb-6 branch August 29, 2023 10:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants