Skip to content

Conversation

@djechlin-mongodb
Copy link
Contributor

Description

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@djechlin-mongodb djechlin-mongodb marked this pull request as ready for review March 6, 2025 15:42
Copy link
Collaborator

@addaleax addaleax left a comment

Choose a reason for hiding this comment

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

Looks good but we really should prioritize the actual bump, and for this PR, and maybe we'd want to leave a TODO comment with a ticket for re-enabling the node-abi package in this script?

Copy link
Collaborator

@gribnoysup gribnoysup left a comment

Choose a reason for hiding this comment

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

Probalby can be a bit more detailed with a comment there, but not a blocker otherwise

@djechlin-mongodb djechlin-mongodb merged commit 1dae5e8 into main Mar 6, 2025
6 of 7 checks passed
@djechlin-mongodb djechlin-mongodb deleted the COMPASS-9075-electron-bumps branch March 6, 2025 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants