Skip to content

Comments

maint(common): Downgrade Crowdin GHA to v1.20.4#15395

Closed
darcywong00 wants to merge 1 commit intomasterfrom
maint/common/crowdin-gha-1-20-4
Closed

maint(common): Downgrade Crowdin GHA to v1.20.4#15395
darcywong00 wants to merge 1 commit intomasterfrom
maint/common/crowdin-gha-1-20-4

Conversation

@darcywong00
Copy link
Contributor

@darcywong00 darcywong00 commented Jan 6, 2026

(updated to include current error)

I think this addresses #15392 (follows #15373 which didn't work)

See #12116, where the Crowdin CLI version 4.0+ is giving language mapping errors.
Log from a recent failing action:

STARTING CROWDIN ACTION
UPLOAD SOURCES
❌ Configuration file is invalid. Check the following parameters in your configuration file:
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.
	- The mapping format is the following: crowdin_language_code: code_you_use. Check the full list of Crowdin language codes that can be used for mapping: https://developer.crowdin.com/language-codes.

Very unfortunate Crowdin doesn't tell us what langauge code it's failing on - even when I run the CLI in verbose mode. 😞

When #12116 gets resolved where we can use Crowdin CLI 4.0+, then we can update the GHA version.

Change

This workaround PR reverts the GitHub action to v1.20.4 which uses CLI 3.19.4.

Test-bot: skip
Build-bot: skip

@darcywong00 darcywong00 added this to the A19S20 milestone Jan 6, 2026
@github-project-automation github-project-automation bot moved this to Todo in Keyman Jan 6, 2026
@keymanapp-test-bot
Copy link

User Test Results

Test specification and instructions

User tests are not required

@github-actions github-actions bot added common/ maint Maintenance work -- continuous integration, build scripts, infrastructure labels Jan 6, 2026
@mcdurdin
Copy link
Member

mcdurdin commented Jan 6, 2026

I am concerned that this is a backwards step - downgrading a dependency/tool needs to be carefully considered because it locks us into an old version which will cause future maintenance headaches. Please do clearly outline the specific reason why we need to have an older version - including error messages and mitigation steps taken with the latest release.

@darcywong00
Copy link
Contributor Author

Please do clearly outline the specific reason why we need to have an older version - including error messages and mitigation steps taken with the latest release.

I've updated the PR description with the error log and additional info

@darcywong00
Copy link
Contributor Author

I'll hold off on merging this to see if it gets replaced by #15407

@darcywong00
Copy link
Contributor Author

This can be abandoned. Replaced by #15407

@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Jan 13, 2026
@darcywong00 darcywong00 deleted the maint/common/crowdin-gha-1-20-4 branch January 15, 2026 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

common/ maint Maintenance work -- continuous integration, build scripts, infrastructure

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants