-
Notifications
You must be signed in to change notification settings - Fork 42
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
provides tooling for interactive localization updates #1348
Conversation
Rename "update-translation-catalogs" to "extract-strings". Remove automatic "git add" operations intended for CI.
f1386ba
to
198262a
Compare
It appears from WeblateOrg/weblate@b965961 that ".mo" files must already be present in the repository for the weblate.gettext.mo add-on to keep them current.
@creviera, this is ready for your review per #1317 (comment) and our conversation this morning. I've divided the test plan into separate sections for the developer experience and the Weblate workflow. The former is worth stepping through as written; but let me know if you'd like to pair on the latter, since it takes place between the Weblate sandbox and my |
I realized today that this pull request has neglected to document the addition of |
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.
this works! exciting to see so many pieces working seamlessly together. nice work, @cfm.
@conorsch - I would merge this now, but want to err on the side of caution until the 0.5.0 release is finalized (would be nice to fix up our release workflow one day so we don't have to halt merging to the client repo during QA but that's for another time!)
(I ended up checking the boxes directly in the test plan in the PR description out of laziness- it all checked out) |
This post-merge step is done— —and https://weblate.securedrop.org/projects/securedrop/securedrop-client/ reset from |
After #1348, setuptools is not responsible for compiling the portable to machine objects, only for including both as produced by Weblate.
After #1348, setuptools is not responsible for compiling the portable to machine objects, only for including both as produced by Weblate.
Description
Closes #1317 by:
l10n
GitHub Actions workflow added in mechanizes continuous-localization workflow #1282;extract-strings
andcheck-strings
for clarity;check-strings
in CI to "lint" that the translation catalog has been updated viaextract-strings
;check-strings
configurable withmake hooks
; andStep (5) is this pull request's addition over "option A" #1347.
How to think about this pull request
The following gist offers background considerations shared with, and for evaluating this "option B" pull request against, "option A" #1347:
https://gist.github.com/cfm/d3173206f55d7e44152928fcfd215031
Test Plan
To test the developer experience
_()
-wrapped source string and commit it1348-cfm-testing
(wherecfm
is your GitHub username ;-)test-buster
fails onmake check-strings
with the following error message:make hooks
(or rerunmake venv
or your platform-appropriate variant)git commit --all
and confirm that the pre-commit hook fails onmake check-strings
as abovemake extract-strings
git commit --all
and confirm that the commit succeedstest-buster
passesTo test the Weblate features
cfm/securedrop-client
git remote add cfm git@github.com:cfm/securedrop-client.git
git push -f cfm 1348-cfm-testing^:i18n-testing
(where1348-cfm-testing
is the testing branch you used above), to push your string change beforemake extract-strings
cfm/securedrop-client
: https://weblate-sandbox.securedrop.org/projects/securedrop/securedrop-client/#repositoryExplore Weblate and the pull request it maintains incfm/securedrop-client
git push cfm 1348-cfm-testing:i18n-testing
(where1348-cfm-testing
is the testing branch you used above), to push your string change withmake extract-strings
cfm/securedrop-client
: https://weblate-sandbox.securedrop.org/projects/securedrop/securedrop-client/#repositorycfm/securedrop-client
Checklist
Before merge
(option B)
from title and close provides tooling for interactive localization updates (option A) #1347 without mergingAfter merge
securedrop-client
Weblate component fromweblate-sandbox
toweblate
:For review
If these changes modify code paths involving cryptography, the opening of files in VMs or network (via the RPC service) traffic, Qubes testing in the staging environment is required. For fine tuning of the graphical user interface, testing in any environment in Qubes is required. Please check as applicable:
If these changes add or remove files other than client code, the AppArmor profile may need to be updated. Please check as applicable:
If these changes modify the database schema, you should include a database migration. Please check as applicable:
main
and confirmed that the migration applies cleanlymain
and would like the reviewer to do so