python312Packages.citeproc-py: Remove nose dependency and re-enable tests#330237
python312Packages.citeproc-py: Remove nose dependency and re-enable tests#330237emilazy merged 2 commits intoNixOS:masterfrom
Conversation
emilazy
left a comment
There was a problem hiding this comment.
Looks like this project might actually be becoming active again. They’ve moved to pynose upstream, but just as a test runner, so using pytestCheckHook should be fine.
There have been a bunch of fix commits since the last release, so maybe we should pin an unstable version until they figure out doing releases again.
There was a problem hiding this comment.
|
(Sorry about all the nitpicking! I really appreciate everyone taking the initiative to deal with this massive list of packages; I just want to make sure we do what we can to get packages in a good state or else get rid of unnecessary burdens in the process.) |
|
No worries! I appreciate the nitpicks, they help improve future packages I submit and/or update! |
8cc35a6 to
371dc89
Compare
371dc89 to
fdc84b4
Compare
emilazy
left a comment
There was a problem hiding this comment.
Result of nixpkgs-review pr 330237 run on x86_64-darwin 1
8 packages built:
- python311Packages.citeproc-py
- python311Packages.citeproc-py.dist
- python311Packages.duecredit
- python311Packages.duecredit.dist
- python312Packages.citeproc-py
- python312Packages.citeproc-py.dist
- python312Packages.duecredit
- python312Packages.duecredit.dist
Description of changes
Part of #326513
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-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.