Skip to content

fix(javascript): enableNpm option have been removed from nixpkgs#2540

Closed
shikanime wants to merge 1 commit into
cachix:mainfrom
shikanime:pr2540
Closed

fix(javascript): enableNpm option have been removed from nixpkgs#2540
shikanime wants to merge 1 commit into
cachix:mainfrom
shikanime:pr2540

Conversation

@shikanime
Copy link
Copy Markdown
Contributor

@shikanime shikanime commented Mar 3, 2026

Fixes: #2538
Signed-off-by: William Phetsinorath william.phetsinorath@shikanime.studio
Change-Id: I3f64d53a63ef26dd285c3575276f10686a6a6964

@domenkozar
Copy link
Copy Markdown
Member

we need to check if it supports the parameter and still call it if it does to support old nixpkgs

Fixes: cachix#2538
Signed-off-by: William Phetsinorath <william.phetsinorath@shikanime.studio>
Change-Id: I3f64d53a63ef26dd285c3575276f10686a6a6964
@shikanime
Copy link
Copy Markdown
Contributor Author

@domenkozar done 👍

@shikanime
Copy link
Copy Markdown
Contributor Author

Thinking about it, npm should not even exist anymore at this point.

@sandydoo
Copy link
Copy Markdown
Member

sandydoo commented Mar 5, 2026

Fixed in 6da2445.

@shikanime, sorry for sniping this one from you, but I needed a quickfix after bumping devenv-nixpkgs.

One issue with the PR here is that default args don't show up in the args you get from .override. This is common footgun we've encounteredd across our modules.

@sandydoo sandydoo closed this Mar 5, 2026
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.

JavaScript failed on NPM because of nodejs-slim nixpkgs changes

3 participants