-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
weblate: 5.8.3 -> 5.9.2 and fix #369307
base: master
Are you sure you want to change the base?
weblate: 5.8.3 -> 5.9.2 and fix #369307
Conversation
edcdb0a
to
c98b4a8
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.
Thanks for the work! I got stuck at making fedora-messaging work in my attempt.
Asking as a person not super familiar with python packaging, is using the GitHub source generally preferred over Pypi tarballs? I always figured it would be the other way round.
Btw feel free to list me as maintainer in the packages you added, I try to maintain Weblate and its dependencies.
|
||
pyproject = true; | ||
|
||
disabled = python.pythonOlder "3.11"; |
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 remove this? Upstream says the same thing https://github.com/WeblateOrg/weblate/blob/main/pyproject.toml#L167C1-L167C27
]; | ||
|
||
pythonImportsCheck = [ "crochet" ]; | ||
|
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.
nativeCheckInputs = [ pytestCheckHook ]; |
I noticed crochet has pytest tests.
Changelog: https://github.com/WeblateOrg/weblate_schemas/blob/2024.2/CHANGES.rst
Changelog: https://github.com/WeblateOrg/translation-finder/blob/2.19/CHANGES.rst
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.