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: Add GitHub action to run translation scripts on push to l10n_develop branch #5919

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

teolemon
Copy link
Member

@teolemon teolemon commented Nov 24, 2024

Add a new GitHub action to run translation scripts on push to l10n_develop branch.

  • Trigger: Set the action to run on push to the l10n_develop branch.
  • Checkout: Add a step to check out the repository.
  • Run Scripts: Add steps to run the iOS translation script and the l10n translation script, including setting execute permissions for the scripts.

For more details, open the Copilot Workspace session.

…lop branch

Add a new GitHub action to run translation scripts on push to `l10n_develop` branch.

* **Trigger**: Set the action to run on push to the `l10n_develop` branch.
* **Checkout**: Add a step to check out the repository.
* **Run Scripts**: Add steps to run the iOS translation script and the l10n translation script, including setting execute permissions for the scripts.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/openfoodfacts/smooth-app?shareId=XXXX-XXXX-XXXX-XXXX).
@g123k
Copy link
Collaborator

g123k commented Nov 24, 2024

It would be nice if this step could generate errors for:

[app_he.arb:prices_proofs_list_length_one_page] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {} two {כל {count} ההוכחות} many {כל {count} ההוכחות}=0{אין הוכחות עדיין} =1{הוכחה אחת בלבד} other{כל {count} ההוכחות}}
                   ^
[app_nl.arb:prices_add_n_prices] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=1{Voeg 1 prijs toe} other{Voeg {count} prijzen toe}}
                   ^
[app_nl.arb:prices_send_n_prices] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=1{Verstuur 1 prijs} other{Verstuur {count} prijzen}}
                   ^
[app_nl.arb:prices_product_accessibility_summary] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=1{1 prijs} other{{count} prijzen}} voor {product}
                   ^
[app_nl.arb:prices_list_length_one_page] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=0{Nog geen prijs} =1{Slechts één prijs} other{Alle {count} prijzen}}
                   ^
[app_nl.arb:prices_proofs_list_length_one_page] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=0{Nog geen bewijs} =1{Slechts één bewijs} other{Alle {count} bewijzen}}
                   ^
[app_pt.arb:prices_add_n_prices] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=1{Adicionar um preço} other{Aplicativo {count} preços}}
                   ^
[app_pt.arb:prices_send_n_prices] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=1{Enviar o preço} other{Aplicativo {count} preços}}
                   ^
[app_pt.arb:prices_product_accessibility_summary] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {}=1{1 preço} other{{count} preços}} para {product}
                   ^
[app_uk.arb:prices_add_n_prices] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {Додати{count} ціну} few {Додати {count} ціни} many {Додати {count} цін}=1{Додати ціну} other{Додати {count} ціни}}
                   ^
[app_uk.arb:prices_send_n_prices] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {Надіслати{count} ціну} few {Надіслати {count} ціни} many {Надіслати {count} цін}=1{Надіслати ціну} other{Надіслати {count} ціни}}
                   ^
[app_uk.arb:prices_product_accessibility_summary] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {{count} ціна} few {{count} ціни} many {{count} цін}=1{1 ціна} other{{count} ціни}} за {product}
                   ^
[app_uk.arb:prices_list_length_one_page] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {Усього {count} ціна} few {Усі {count} ціни} many {Усі {count} ціни}=0{Ціни ще немає} =1{Тільки одна ціна} other{Усі {count} ціни}}
                   ^
[app_uk.arb:prices_proofs_list_length_one_page] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    {count,plural, one {Усього {count} доказ} few {Усі {count} докази} many {Усі {count} докази}=0{Доказів ще немає} =1{Тільки один доказ} other{Усі {count} докази}}
                   ^
[app_uk.arb:robotoff_next_n_questions] ICU Syntax Warning: The plural part specified below is overridden by a later plural part.
    Наступне {count,plural,  one {{count} запитання}  few {{count} запитання}  many {{count} запитань}=1{запитання} other{{count} запитання}}
                             ^

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.

@teolemon teolemon merged commit 514fda9 into develop Nov 25, 2024
9 of 10 checks passed
@teolemon teolemon deleted the teolemon/add-run-translation-scripts branch November 25, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants