mirror of
https://github.com/CringeStudios/element-desktop.git
synced 2025-01-18 23:44:59 +01:00
Consolidate i18n check into a reusable workflow (#361)
* Fix i18n check bypass for RiotTranslateBot * Consolidate i18n check into a reusable workflow
This commit is contained in:
parent
2a2e6781c1
commit
bce69efd37
36
.github/workflows/static_analysis.yaml
vendored
36
.github/workflows/static_analysis.yaml
vendored
@ -23,41 +23,7 @@ jobs:
|
|||||||
|
|
||||||
i18n_lint:
|
i18n_lint:
|
||||||
name: "i18n Check"
|
name: "i18n Check"
|
||||||
runs-on: ubuntu-latest
|
uses: matrix-org/matrix-react-sdk/.github/workflows/i18n_check.yml@develop
|
||||||
permissions:
|
|
||||||
pull-requests: read
|
|
||||||
steps:
|
|
||||||
- uses: actions/checkout@v2
|
|
||||||
|
|
||||||
- name: "Get modified files"
|
|
||||||
id: changed_files
|
|
||||||
if: github.event_name == 'pull_request' && github.actor != 'RiotTranslateBot'
|
|
||||||
uses: tj-actions/changed-files@v19
|
|
||||||
with:
|
|
||||||
files: |
|
|
||||||
src/i18n/strings/*
|
|
||||||
files_ignore: |
|
|
||||||
src/i18n/strings/en_EN.json
|
|
||||||
|
|
||||||
- name: "Assert only en_EN was modified"
|
|
||||||
if: |
|
|
||||||
github.event_name == 'pull_request' &&
|
|
||||||
github.actor != 'RiotTranslateBot' &&
|
|
||||||
steps.changed_files.outputs.any_modified == 'true'
|
|
||||||
run: |
|
|
||||||
echo "You can only modify en_EN.json, do not touch any of the other i18n files as Weblate will be confused"
|
|
||||||
exit 1
|
|
||||||
|
|
||||||
- uses: actions/setup-node@v3
|
|
||||||
with:
|
|
||||||
cache: 'yarn'
|
|
||||||
|
|
||||||
# Does not need branch matching as only analyses this layer
|
|
||||||
- name: Install Deps
|
|
||||||
run: "yarn install --pure-lockfile"
|
|
||||||
|
|
||||||
- name: i18n Check
|
|
||||||
run: "yarn run diff-i18n"
|
|
||||||
|
|
||||||
js_lint:
|
js_lint:
|
||||||
name: "ESLint"
|
name: "ESLint"
|
||||||
|
Loading…
Reference in New Issue
Block a user