Page MenuHomePhorge
Differential D196

Add CI to validate ftls and ensure core languages are up-to-date
ClosedPublic
Actions

Authored by tusooa on Sat, Nov 2, 2:23 PM.

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rK51f963f621f9: Add CI to validate ftls and ensure core languages are up-to-date
Summary

This ensures translation files are all valid, and translations in all core languages are up-to-date with each other. Unit tests of the lint util itself is also added.

Type: skip

Test Plan

Verify pipeline passes.

Diff Detail

Repository
rK kazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 314
Build 616: GitLab CI for kazv
Build 615: arc lint + arc unit

Event Timeline

tusooa created this revision.Sat, Nov 2, 2:23 PM
Owners added a reviewer: O1: the Kazv Project.Sat, Nov 2, 2:23 PM
tusooa mentioned this in D194: Remove useless items and added missing items of create-room-page series.Sat, Nov 2, 2:25 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Sat, Nov 2, 2:38 PM
Harbormaster failed remote builds in B311: Diff 519!
tusooa updated this revision to Diff 520.Sat, Nov 2, 2:48 PM

Fix inconsistency of cmn-Hans translation key

Harbormaster completed remote builds in B312: Diff 520.Sat, Nov 2, 3:03 PM
tusooa requested review of this revision.Sat, Nov 2, 3:03 PM
tusooa updated this revision to Diff 522.Sat, Nov 2, 9:31 PM

Run pipeline for all js and mjs file changes

Harbormaster completed remote builds in B314: Diff 522.Sat, Nov 2, 9:47 PM
nannanko accepted this revision.Sat, Nov 9, 4:04 AM
This revision is now accepted and ready to land.Sat, Nov 9, 4:04 AM
Closed by commit rK51f963f621f9: Add CI to validate ftls and ensure core languages are up-to-date. · Explain WhySat, Nov 9, 6:01 PM
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rK51f963f621f9: Add CI to validate ftls and ensure core languages are up-to-date.

Revision Contents
Changeset List

Diff 522

View Options

.gitlab-ci.yml

Loading...
View Options

README.md

Loading...
View Options

src/js/l10n-config.json

Loading...
View Options

src/js/lint-l10n-lib.mjs

Loading...
View Options

src/js/lint-l10n.mjs

Loading...
View Options

src/js/package-lock.json

Loading...
View Options

src/js/package.json

Loading...
View Options

src/js/test/fixtures/2same1diff/x-a/0.ftl

Loading...
View Options

src/js/test/fixtures/2same1diff/x-b/0.ftl

Loading...
View Options

src/js/test/fixtures/2same1diff/x-b/1.ftl

Loading...
View Options

src/js/test/fixtures/2same1diff/x-c/0.ftl

Loading...
View Options

src/js/test/fixtures/empty/x-foo/0.ftl

Loading...
View Options

src/js/test/fixtures/invalid-concat-valid/x-foo/0.ftl

Loading...
View Options

src/js/test/fixtures/invalid-concat-valid/x-foo/1.ftl

Loading...
View Options

src/js/test/fixtures/invalid/x-foo/0.ftl

Loading...
View Options

src/js/test/test.mjs

Loading...
View Options

src/l10n/cmn-Hans/100-ui.ftl

Loading...