Skip to content

Fix transifex translation handling #1560

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.

Sign up for GitHub

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

Jump to bottom
Closed
umbynos opened this issue Nov 22, 2021 · 0 comments · Fixed by #1578
Closed

Fix transifex translation handling #1560

umbynos opened this issue Nov 22, 2021 · 0 comments · Fixed by #1578
Assignees
@silvanocerza
Labels
conclusion: resolved Issue was resolved topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project

Comments

@umbynos
Copy link
Contributor

umbynos commented Nov 22, 2021

Currently, the translation handling has a problem:
Between the push to transifex and the pull of the translation files, the transifex service do some magic changing them. This is an issue because the task i18:check would fail when releasing the cli. A solution could be to modify the main.go file responsible for the push to transifex in order to use the same format
@umbynos umbynos added topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project labels Nov 22, 2021
@silvanocerza silvanocerza mentioned this issue Dec 2, 2021
Merged
5 tasks
@per1234 per1234 added the conclusion: resolved Issue was resolved label Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@silvanocerza silvanocerza

Labels
conclusion: resolved Issue was resolved topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enhance i18n:check task arduino/arduino-cli
3 participants
@silvanocerza @per1234 @umbynos