-
-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
nodePackages: update to latest #279469
nodePackages: update to latest #279469
Conversation
943a0b5
to
79a5ecd
Compare
79a5ecd
to
331c2b3
Compare
Rebased due to conflict. Quick question, what's the usual process to get something merged? |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1395 |
Please extract it from nodePackages and convert it to buildNodePackage |
Result of 6 packages failed to build:
61 packages built:
|
Indeed. See #229475. |
I looked into it, and it seems like it's already been done, just not removed from the nodePackages. I'll clean that up |
Description of changes
Just wanted to get some updates for an lsp server I'm using, took the opportunity to get everything since it's a bit more straightforward.
Generated using
/pkgs/development/node-packages/generate.sh
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.