-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
manga-tui: 0.3.1 -> 0.4.0 #353060
manga-tui: 0.3.1 -> 0.4.0 #353060
Conversation
PR got merged upstream so I'm just applying the commit as a patch until the next release. |
|
This test seems flaky and non deterministic, it builds fine for me but seems to fail sometimes. I'm gonna open an issue upstream. Until then we can just patch the test out. |
0f6d323
to
ed2f0d7
Compare
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.
nixpkgs-review
result
Generated using nixpkgs-review
.
Command: nixpkgs-review pr 353060
x86_64-linux
✅ 1 package built:
- manga-tui
Thank you and welcome to nixpkgs! Now that you've added yourself to |
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-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.
There are issues with upstream when building in Nix (specifically, there's 2 tests that don't pass). I submitted a PR upstream to fix one of them but am not sure how to fix the other, so I just added a patch to delete it for the time being.