-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat: Add MongoOption builder logic #2623
Merged
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
5b0212d
chore: remove old option usages from tests
nbbeeken 85ec4b5
revert accidental option removal
nbbeeken 658ce1c
make diff smaller
nbbeeken 128ab76
feat: :art: Add MongoOptions interface
nbbeeken 6a58338
fix failing tests, silence error
nbbeeken 392b7f8
lint
nbbeeken de0f364
Pick options off of MongoClientOptions
nbbeeken 72c11cf
Clearer ternary statement
nbbeeken 23e9394
Move around code
nbbeeken 2c90aa7
feat: Add MongoOptions builder
nbbeeken 9e73e34
More tests
nbbeeken 77e0b30
Remove unused or alias options
nbbeeken 1fe9412
test fixes
nbbeeken 3474448
import fix
nbbeeken 9504c67
Improved spec testing
nbbeeken b8d984e
linter fixes
nbbeeken 3ccf21b
fix: remove Enum suffix
nbbeeken ebe95ee
fix: Auth mechanism use consolidation
nbbeeken f226e4b
fixes
nbbeeken bb935b6
fixed tests
nbbeeken a490ad9
Improve isRecord and remove alias options
nbbeeken 79039e2
Move isRecord to utils and remove deprecated opt
nbbeeken ae88b27
Fix up specialized object type construction
nbbeeken c8daecf
TS fix
nbbeeken cb27c3d
Reduce casing complexity, loop over options registery, endsWith('sock')
nbbeeken fe8dffd
rebase fixes
nbbeeken File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mbroadst
Sorry about my git fu, I can't reply to your comment about the credentials issue. I've gone ahead with the merge solution, lmk what you think. For what its worth I think we were safe with the spread solution because typescript would not permit us to have a property missing, it just so happens that MongoCredentials and MongoCredentialOptions share all their properties and mandate that they are defined, but ofc better to be safe and box up more complex functionality, so
.merge
seems like a good solution .