-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
python310Packages.flask-wtf: fix failing build #190680
python310Packages.flask-wtf: fix failing build #190680
Conversation
Pull request opened on the Flask WTF repository: pallets-eco/flask-wtf#533 |
@ofborg build calibre-web |
Result of 3 packages marked as broken and skipped:
5 packages failed to build:
10 packages built:
|
@ofborg build pgadmin4 python310Packages.flask-admin python310Packages.ihatemoney apache-airflow python310Packages.apache-airflow |
pgadmin build will be fixed with #188575 6.13 requires Werkzeug 2.1 |
Arguably you could just pull your fix as a patch, we may forget to ever remove the test disablement. |
@@ -32,10 +32,14 @@ buildPythonPackage rec { | |||
pytestCheckHook | |||
]; | |||
|
|||
disabledTests = [ | |||
"test_set_default_message_language" |
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.
Please add a comment why this test is disabled
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.
Arguably you could just pull your fix as a patch, we may forget to ever remove the test disablement.
Thanks for the review.
I removed this disabled test and use the fix as a patch in the package.
0d05cb3
to
7083952
Compare
Description of changes
Fix failing build of flask-wtf
Added myself to maintainers
Fixes the flask-wtf failing build on the staging-next PR/branch: #188284
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes