[WIP] Update nodePackages with node2nix 1.10.0#146440
[WIP] Update nodePackages with node2nix 1.10.0#146440svanderburg wants to merge 8 commits intoNixOS:masterfrom
Conversation
|
Great to see you around! (Thanks a lot for the node2nix contribution btw) |
|
@happysalada Yes I've added an option to handle Node.js 16.x out of the box. Some of the stuff that I need seems to work, but we still have to see how well the integration works. I also intend make the modifications so that we can package for Node.js 16.x. We probably should start with a small set of packages first and then gradually expand it. |
|
Amazing thanks! I think prettier was one of the problematic packages. If it works properly for prettier, then the potential problem might have been fixed. |
|
Hi. Is there anything blocking updating to node2nix 1.10.0? |
|
What's blocking this? |
|
I'm wondering why But: It doesn't even show the |
That's because of the override in |
|
My suggestion would be something like this where we don't touch packages that aren't inside |
|
I believe this is blocking fixing mongosh and pretty much all nodejs_16x packages. It seems like it was approved but never merged, so this should be low-hanging fruit if the merge conflict isn't too difficult? |
node2nix was already updated in #171195 as I understand it. This PR also regenerates a few of the packages not under I believe there are still some issues with node2nix and Node.js 16+ that will need to be fixed in svanderburg/node2nix first before some of those packages will work correctly. Mostly svanderburg/node2nix#236 (comment) and svanderburg/node2nix#293 I think |
I don't think this is true -- node2nix still cannot output v2 lockfiles. |
My mistake, I'm new to node/javascript on NixOS and just got here by attempting to connect to a mongo server using the mongo cli on nixos 😅 |
Motivation for this change
I'm working on updating the NPM packages to be regenerated with node2nix 1.10.0
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)