Skip to content
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

ci(translation-tests): change filter prefix s/i18n/weblate/ #6985

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

cfm
Copy link
Member

@cfm cfm commented Oct 12, 2023

Status

Ready for review

Description of Changes

After #6954, runs translation-tests on Weblate's pull requests prefixed weblate- rather than our custom branches prefixed i18n-.

Testing

  • CI on this branch includes translation-tests.

Deployment

No special considerations; CI-only.

Weblate opens pull requests from
weblate-fpf/securedrop@weblate-securedrop-securedrop.
@cfm cfm added this to the SecureDrop 2.7.0 milestone Oct 12, 2023
@cfm cfm requested a review from a team as a code owner October 12, 2023 01:03
Copy link
Member

@legoktm legoktm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As described your PR accomplishes that so giving it my approval, but I guess I misinterpreted #6876 in thinking that we'd run translation-tests on every PR? Is there a reason we don't? Doing so would also be in line with #6980.

(Feel free to merge if you want to pick this up as a small change for now and leave dropping the branch filter entirely for the other ticket)

@cfm
Copy link
Member Author

cfm commented Oct 12, 2023

That's my inclination, @legoktm. translation-tests is 21 languages × ~12 minutes = ~252 CI minutes a pop. That's justified for incoming translations, but I'm reluctant to do that for incoming everything until #6980.

@cfm cfm merged commit c8d6b8f into develop Oct 12, 2023
@legoktm legoktm deleted the weblate-test-prefix branch October 12, 2023 18:13
@cfm
Copy link
Member Author

cfm commented Oct 12, 2023

Actually, while debugging https://app.circleci.com/pipelines/github/freedomofpress/securedrop/6416/workflows/86ef51ed-fb3d-49c1-9f87-96d59e19dd2f/jobs/72712, I've realized this is wrong anyway:

circleci@ip-10-0-25-2:~/project$ git branch 
  develop
* pull/6984
circleci@ip-10-0-25-2:~/project$ echo $CIRCLE_BRANCH
pull/6984
circleci@ip-10-0-25-2:~/project$ echo $CIRCLE_PR_USERNAME
weblate-fpf

So I'll have a fix for this fix shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants