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
pythonPackages.rq: init at 1.1.0 #64525
Conversation
889ebe5
to
d81c7ea
Compare
Added tests and quoted the homepage URL |
sha256 = "0rx8xji473ggmcy40v7lh0a69j4nrb4qrp4kp1766qg0swp37pi2"; | ||
}; | ||
|
||
checkInputs = [ pytest ]; |
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.
For me at least, this finds zero tests, and looking into this project's tests, they appear to heavily depend on having a running redis server (https://github.com/rq/rq/blob/d0884be9296cf2cba2428d5d90cdeda65a550296/run_tests), which is something we can't do. So you're probably better off adding doCheck = false
and a comment of something like "tests require a running redis server".
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.
Oh, I see now... Updated according to your suggestion 🙏
}; | ||
|
||
checkInputs = [ pytest ]; | ||
buildInputs = [ mock ]; |
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.
Again, this doesn't appear necessary to me.
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.
Sorry! I wasn't quite sure, what does this line do, but thought that it's necessary 🤦♂️
637edf8
to
e46a849
Compare
removed |
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.
Cool 😎
Works for me, non-nixos linux x86_64
Will be good to have rq
.
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.
https://github.com/rq/rq/releases/tag/v1.1.0 was released.
Can you also format your commit like
pythonPackages.rq: init at 1.0
to match the contributing guidelines?
Bumped the version to 1.1.0 and updated the version number in the commit message. |
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.
LGTM 👍
Motivation for this change
Adding a new package.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)