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
flexget: fix build #62258
flexget: fix build #62258
Conversation
sha256 = "151bd816817e645e9064c354b13544f85137ea3320ca3be1fb6873ea75ef7dc1"; | ||
}; | ||
|
||
# No tests in archive |
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.
Why not have tests from github like before?
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.
Fixes build 👍
}; | ||
|
||
# No tests in archive | ||
doCheck = false; |
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.
Isn't it possible to use the GitHub archive here to have a running test suite?
Lol, it's the hive mind of python packages maintainers who really like to enable tests 😄 |
Ouch, seems as my browser is really trolling me today, sorry for the duplicated comment... |
I have that same problem. Usually hard refreshing does the trick. |
Just as question: Since there's already flexget 2.21.2, wouldn't it be better to also use that? |
"zxcvbn-python" was deprecated, and renamed to "zxcvbn". See: dwolfhub/zxcvbn-python#30
@worldofpeace, @Ma27 I've updated |
looks good to me. |
Thanks for fixing this @marsam ✨ Likewise to everyone. |
Motivation for this change
Flexget uses
zxcvbn-python
instead ofzxcvbn
. Fixes #62110cc: @sjau
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)