-
Notifications
You must be signed in to change notification settings - Fork 353
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
Problem: nixpkgs is not pinned properly #235
Conversation
the ledger test fails @linfeng-crypto ?
|
Solution: - use niv to manage nixpkgs and other dependencies - refactor nix expression a little bit
I think it's my fault, it's running with the chain-maind without ledger-zemu. |
Codecov Report
@@ Coverage Diff @@
## master #235 +/- ##
=======================================
Coverage 27.17% 27.17%
=======================================
Files 32 32
Lines 5773 5773
=======================================
Hits 1569 1569
Misses 4021 4021
Partials 183 183
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@@ -146,8 +146,8 @@ clean-docker-compose: localnet-stop | |||
############################################################################### | |||
# nix installation: https://nixos.org/download.html | |||
nix-integration-test: check-network | |||
nix-shell integration_tests/shell.nix --arg ci true --run "pytest -v -n 6 -m 'not ledger' --dist loadscope" | |||
nix-shell integration_tests/shell.nix --arg ci true --run "pytest -v -m ledger" | |||
nix-shell ./. -A ci-shell --run "pytest -v -n 7 -m 'not ledger' --dist loadscope" |
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.
what's meaning of ./.?
if it means current folder, how about $PWD for readibility?
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's short for ./default.nix
, I guess you can also use $PWD/.
nixpkgs-fmt | ||
]; | ||
} | ||
run: nix-shell ./. -A lint-shell --run "make lint-ci" |
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.
how about nix-shell $PWD?
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
nixpkgs is only pinned with environment variable, better to use niv to manage.
Solution: