Skip to content

fix: handle missing node-gyp gracefully in @npmcli/config definitions - #8650

Merged
owlstronaut merged 1 commit into
latestfrom
config-no-node-require
Oct 6, 2025
Merged

fix: handle missing node-gyp gracefully in @npmcli/config definitions#8650
owlstronaut merged 1 commit into
latestfrom
config-no-node-require

Conversation

@owlstronaut

@owlstronautowlstronaut commented Oct 6, 2025

Copy link
Copy Markdown

fixes#8581

It makes sense that npm config shouldn't necessarily require node-gyp to be present.

@owlstronaut
owlstronaut requested a review from a team as a code ownerOctober 6, 2025 21:45
@owlstronaut
owlstronaut merged commit 5b4a7fc into latestOct 6, 2025
17 checks passed
@owlstronaut
owlstronaut deleted the config-no-node-require branch October 6, 2025 22:10
@github-actionsgithub-actionsBot mentioned this pull request Oct 6, 2025
@github-actionsgithub-actionsBot mentioned this pull request May 27, 2026
Sign up for freeto 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.

[BUG] @npmcli/config package has implicit dependency on node-gyp

2 participants

@owlstronaut@wraithgar