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

feat(NODE-6274): add CSOT support to bulkWrite #4250

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

nbbeeken
Copy link
Contributor

Description

What is changing?

  • Adds CSOT support to bulkWrite
Is there new documentation needed for these changes?

No

What is the motivation for this change?

CSOT for all operations

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@nbbeeken nbbeeken force-pushed the NODE-6274-bulkCSOT branch 2 times, most recently from d463822 to 416ee1a Compare September 27, 2024 20:22
@nbbeeken nbbeeken marked this pull request as ready for review September 27, 2024 20:22
@W-A-James W-A-James self-assigned this Sep 30, 2024
@W-A-James W-A-James self-requested a review September 30, 2024 14:51
@W-A-James W-A-James added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Sep 30, 2024
@nbbeeken nbbeeken requested a review from W-A-James September 30, 2024 17:54
@W-A-James W-A-James added Team Review Needs review from team and removed Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Sep 30, 2024
@aditi-khare-mongoDB aditi-khare-mongoDB self-requested a review October 1, 2024 15:05
@W-A-James
Copy link
Contributor

Note: Test failures here are a result of known and unrelated KMS server issue which will be addressed in NODE-6405

@W-A-James W-A-James merged commit 09f6d7d into NODE-6090 Oct 2, 2024
19 of 27 checks passed
@W-A-James W-A-James deleted the NODE-6274-bulkCSOT branch October 2, 2024 21:32
baileympearson added a commit that referenced this pull request Oct 10, 2024
baileympearson added a commit that referenced this pull request Oct 10, 2024
nbbeeken added a commit that referenced this pull request Oct 14, 2024
baileympearson added a commit that referenced this pull request Oct 21, 2024
nbbeeken added a commit that referenced this pull request Nov 1, 2024
nbbeeken added a commit that referenced this pull request Nov 5, 2024
dariakp pushed a commit that referenced this pull request Nov 6, 2024
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.

4 participants