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

fix: add back bin/node-gyp-bin/node-gyp files #6932

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

lukekarrys
Copy link
Contributor

This was an unintended breaking change as part of #6554. The node-gyp
bin files are not used by the CLI directly but they are relied on by
other tooling. This change is only intended to land on the v9 release
line.

This partially reverts commit 3a7378d.

@lukekarrys lukekarrys requested a review from a team as a code owner October 25, 2023 16:49
This was an unintended breaking change as part of #6554. The `node-gyp`
bin files are not used by the CLI directly but they are relied on by
other tooling. This change is only intended to land on the v9 release
line.

This partially reverts commit 3a7378d.
Copy link
Member

@wraithgar wraithgar left a comment

Choose a reason for hiding this comment

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

Land it when it's green.

@lukekarrys lukekarrys merged commit c93edb5 into release/v9 Oct 26, 2023
28 checks passed
@lukekarrys lukekarrys deleted the lk/node-gyp-bin-revert branch October 26, 2023 00:29
@github-actions github-actions bot mentioned this pull request Oct 26, 2023
lukekarrys added a commit that referenced this pull request Oct 30, 2023
This is a continuation of #6932. This makes a few more changes
identified in also porting these changes to v10.
lukekarrys added a commit that referenced this pull request Oct 30, 2023
This is a continuation of #6932. This makes a few more changes
identified in also porting these changes to v10.
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.

2 participants