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
pass-git-helper: 1.1.0 -> 1.1.1 #108513
pass-git-helper: 1.1.0 -> 1.1.1 #108513
Conversation
pkgs/applications/version-management/git-and-tools/pass-git-helper/default.nix
Outdated
Show resolved
Hide resolved
Getting this error when building. do you know off-hand if that's a new dependency on your side?
|
NVM here's something that builds afaict: |
Sorry for the delay and thanks for the help. I have updated the commit. |
pkgs/applications/version-management/git-and-tools/pass-git-helper/default.nix
Outdated
Show resolved
Hide resolved
The |
Bump package and add new test requirements.
Rebased |
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.
- Tested compilation of all pkgs that depend on this change using
nixpkgs-review
1 package built:
pass-git-helper
- Tested execution of all binary files (one)
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
}; | ||
|
||
propagatedBuildInputs = [ pyxdg ]; | ||
checkInputs = [ pytest ]; | ||
checkInputs = [ pytest pytest-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.
It might be worth it to move pytestCheckHook.
Motivation for this change
I have released a new version upstream of the package. As I don't use nix and have no tools available, I don't know how to compute and updated sha256sum for this. If someone could help, that would be great.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)