Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(NODE-2998): move connection-string parsing into separate package #2848

Merged
merged 1 commit into from
Jun 25, 2021

Conversation

addaleax
Copy link
Contributor

Description

What changed?

As discussed earlier this year, this moves the connection string
parsing into a separate package for mongodb-connection-string-specific
URL parsing. The code in that package is based very closely on the
original code in the driver here.

Are there any files to ignore?

No.

As discussed earlier this year, this moves the connection string
parsing into a separate package for mongodb-connection-string-specific
URL parsing. The code in that package is based very closely on the
original code in the driver here.
@dariakp dariakp self-requested a review June 24, 2021 20:11
@dariakp dariakp added Primary Review In Review with primary reviewer, not yet ready for team's eyes Team Review Needs review from team and removed Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Jun 24, 2021
Copy link
Contributor

@nbbeeken nbbeeken left a comment

Choose a reason for hiding this comment

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

Thanks for this! LGTM

Copy link
Contributor

@emadum emadum left a comment

Choose a reason for hiding this comment

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

LGTM

@dariakp dariakp merged commit ac4aabc into mongodb:4.0 Jun 25, 2021
@addaleax addaleax deleted the 2998-dev branch June 30, 2021 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team Review Needs review from team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants