Skip to content

feat(NODE-7403): enable more build warnings on Windows#305

Merged
durran merged 1 commit intomongodb-js:mainfrom
rzhao271:rzhao271/patch-binding
Jan 27, 2026
Merged

feat(NODE-7403): enable more build warnings on Windows#305
durran merged 1 commit intomongodb-js:mainfrom
rzhao271:rzhao271/patch-binding

Conversation

@rzhao271
Copy link
Contributor

@rzhao271 rzhao271 commented Jan 21, 2026

Description

Summary of Changes

This PR enables more build warnings on Windows and is a follow-up to #190.

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

@rzhao271 rzhao271 requested a review from a team as a code owner January 21, 2026 23:57
@rzhao271 rzhao271 force-pushed the rzhao271/patch-binding branch from 8a8a189 to 87a7ac4 Compare January 22, 2026 00:00
@tadjik1 tadjik1 self-assigned this Jan 27, 2026
@tadjik1 tadjik1 added Primary Review In Review with primary reviewer, not yet ready for team's eyes tracked-in-jira labels Jan 27, 2026
Copy link
Contributor

@tadjik1 tadjik1 left a comment

Choose a reason for hiding this comment

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

Thanks @rzhao271, additional options look good!

@tadjik1 tadjik1 requested a review from a team January 27, 2026 11:46
@tadjik1 tadjik1 added Team Review Needs review from team and removed Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Jan 27, 2026
@durran durran self-assigned this Jan 27, 2026
@durran durran merged commit d3e2cb2 into mongodb-js:main Jan 27, 2026
21 checks passed
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 tracked-in-jira

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants