Skip to content

chore(NODE-7759): pin bson-compat server - #5018

Merged
dariakp merged 3 commits into
mainfrom
pin-bson-compat-server
Aug 10, 2026
Merged

chore(NODE-7759): pin bson-compat server#5018
dariakp merged 3 commits into
mainfrom
pin-bson-compat-server

Conversation

@PavelSafronov

@PavelSafronov PavelSafronov commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Description

Summary of Changes

Pin server to a known working version, and add documentation around this variant.

What is the motivation for this change?

Pin the server being used for BSON compat tests, so we are only testing one variable: the latest BSON library.

Double check the following

  • Lint is passing (npm run check:lint)
  • 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

@PavelSafronov
PavelSafronov marked this pull request as ready for review August 6, 2026 19:58
@PavelSafronov
PavelSafronov requested a review from a team as a code owner August 6, 2026 19:58
Copilot AI review requested due to automatic review settings August 6, 2026 19:58
@PavelSafronov PavelSafronov changed the title chore: pin bson-compat server to 8.0.26 chore(NODE-7759): pin bson-compat server to 8.0.26 Aug 6, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@PavelSafronov PavelSafronov changed the title chore(NODE-7759): pin bson-compat server to 8.0.26 chore(NODE-7759): pin bson-compat server Aug 6, 2026
@johnmtll johnmtll self-assigned this Aug 7, 2026
@johnmtll johnmtll added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Aug 7, 2026

@johnmtll johnmtll left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@johnmtll johnmtll added Team Review Needs review from team and removed Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Aug 7, 2026
@dariakp
dariakp merged commit dce7939 into main Aug 10, 2026
30 checks passed
@dariakp
dariakp deleted the pin-bson-compat-server branch August 10, 2026 20:36
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