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

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

Details

Reviewers

None

Group Reviewers

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 311
Build 610: GitLab CI for kazv
Build 609: 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!

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

Run pipeline for all js and mjs file changes

Harbormaster completed remote builds in B314: Diff 522.Sat, Nov 2, 9:47 PM

Revision Contents
Changeset List

Path

Size

Packages

22 lines

10 lines

src/

js/

6 lines

180 lines

36 lines

3756 lines

8 lines

test/

fixtures/

2same1diff/

x-a/

5 lines

x-b/

1 line

4 lines

x-c/

5 lines

empty/

x-foo/

0.ftl

O1: the Kazv Project

invalid-concat-valid/

x-foo/

1 line

1 line

invalid/

x-foo/

1 line

89 lines

Diff 519

View Options

.gitlab-ci.yml

View Options

README.md

View Options

src/js/l10n-config.json

View Options

src/js/lint-l10n-lib.mjs

View Options

src/js/lint-l10n.mjs

View Options

src/js/package-lock.json

View Options

src/js/package.json

View Options

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

View Options

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

View Options

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

View Options

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

View Options

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

View Options

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

View Options

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

View Options

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

View Options

Add CI to validate ftls and ensure core languages are up-to-dateNeeds ReviewPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 519

.gitlab-ci.yml

README.md

src/js/l10n-config.json

src/js/lint-l10n-lib.mjs

src/js/lint-l10n.mjs

src/js/package-lock.json

src/js/package.json

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

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

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

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

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

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

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

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

src/js/test/test.mjs

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

Revision Contents
Changeset List