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
Add the missing pyjson5 module #72974
Conversation
Please check the contributing guidelines for how to name commits. Merges should be avoided so please rebase instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit name should read "pythonPackages.pyjson5: init at 0.8.5"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the commit history should look like:
maintainers: add isgy
python3Packages.pyjson5: init at 0.8.5
Thanks for the fix! Should the derivation for jupyterlab_server be updated as well to add the dependency on pyjson5? |
that is what the PR states :) |
@jonringer @veprbl I'm not sure why the tests are failing (since I haven't touched any of the files in that list) |
@GrahamcOfBorg eval |
The error wasn't even from your package, that was weird |
Always rebase, do not merge master. |
sorry, I know.. fixing |
please merge the two maintainer commits..
|
maintainers: fixed typo
@jonringer @FRidh sorry.. good idea :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nix-review
passes on NixOS
diff LGTM
commits LGTM
failures are broken on master
[6 built (2 failed), 3 copied (9.5 MiB), 9.2 MiB DL]
error: build of '/nix/store/5981hvalbm7s5kgz1sm4wwcd6c5k7pan-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/72974
2 package failed to build:
python37Packages.jupyterlab python38Packages.jupyterlab
5 package were build:
python27Packages.pyjson5 python37Packages.jupyterlab_server python37Packages.pyjson5 python38Packages.jupyterlab_server python38Packages.pyjson5
Motivation for this change
pyjson5 is needed by jupyterlab_server #72965
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @FRidh @jonringer