Skip to content

Commit 69c1d70

Browse files
committed
improving sync when no pr is necessary
1 parent 0e01cac commit 69c1d70

1 file changed

Lines changed: 41 additions & 21 deletions

File tree

.github/workflows/sync_translations.yml

Lines changed: 41 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
steps:
2020
- uses: actions/checkout@v6
2121
with:
22-
token: ${{ secrets.MACHINE_ACCOUNT_PAT }}
22+
token: ${{ secrets.GITHUB_TOKEN }}
2323
ref: 'main'
2424
fetch-depth: 0
2525

@@ -38,7 +38,7 @@ jobs:
3838

3939
- name: Get second to latest run time
4040
env:
41-
GH_TOKEN: ${{ secrets.MACHINE_ACCOUNT_PAT }}
41+
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
4242
run: |
4343
OUTPUT=$(gh run list --workflow sync_translations.yml --json updatedAt | jq -r '.[1].updatedAt')
4444
echo "LATEST_RUN=$OUTPUT" >> $GITHUB_ENV
@@ -69,49 +69,69 @@ jobs:
6969
yarn build
7070
7171
- name: Push translation sources to crowdin
72+
continue-on-error: true
7273
run: |
7374
cd ./tools/localization
7475
yarn start upload
7576
7677
- name: Pull translation updates from crowdin
78+
continue-on-error: true
7779
run: |
7880
cd ./tools/localization
7981
yarn start download
8082
8183
- name: Extract downloaded ankidroid.zip file
84+
continue-on-error: true
8285
run: |
8386
cd ./tools/localization
8487
yarn start extract
8588
8689
- name: Update translation to AnkiDroid res
90+
continue-on-error: true
8791
run: |
8892
cd ./tools/localization
8993
yarn start update
9094
91-
- name: Commit changes
92-
run: |
93-
git add docs/marketing/localized_description AnkiDroid/src/main/res
94-
git commit -am 'Updated strings from Crowdin'
95-
git push --set-upstream origin +i18n_sync
96-
echo "The results of the sync are on the i18n_sync branch, PR them from there for merge."
97-
echo "https://github.com/ankidroid/Anki-Android/compare/i18n_sync?expand=1"
98-
99-
- name: Check if there are strings changes
95+
- name: Check for changes and handle Neutral result
10096
id: tr_check
101-
run: |
102-
git checkout i18n_sync
103-
COMMITS_COUNT=`git rev-list --count HEAD ^main`
104-
if [ "$COMMITS_COUNT" -gt 0 ]; then
105-
echo "HAS_CHANGES=true" >> $GITHUB_OUTPUT
106-
else
107-
echo "HAS_CHANGES=false" >> $GITHUB_OUTPUT
108-
fi
97+
uses: actions/github-script@v7
98+
with:
99+
script: |
100+
const { execSync } = require('child_process');
101+
102+
execSync('git add docs/marketing/localized_description AnkiDroid/src/main/res');
103+
104+
const status = execSync('git diff --cached --name-only').toString().trim();
105+
106+
if(!status) {
107+
console.log("No changes detected. Setting status to neutral.");
108+
109+
await github.rest.checks.create({
110+
owner: context.repo.owner,
111+
repo: context.repo.repo,
112+
name: 'Sync Translations with Crowdin',
113+
head_sha: context.sha,
114+
status: 'completed',
115+
conclusion: 'neutral',
116+
output: {
117+
title: 'Translation Sync',
118+
summary: 'No translation changes were found on Crowdin to sync'
119+
}
120+
});
121+
122+
core.setOutput('HAS_CHANGES', 'false');
123+
} else {
124+
core.setOutput('HAS_CHANGES', 'true');
125+
execSync("git commit -m 'Updated strings from Crowdin'");
126+
execSync("git push --set-upstream origin +i18n_sync");
127+
console.log("Changes pushed to i18n_sync branch");
128+
}
109129
110130
- name: Create PR for strings changes if needed
111-
if: ${{ steps.tr_check.outputs.HAS_CHANGES }}
131+
if: steps.tr_check.outputs.HAS_CHANGES == 'true'
112132
uses: actions/github-script@v9
113133
with:
114-
github-token: ${{ secrets.MACHINE_ACCOUNT_PAT }}
134+
github-token: ${{ secrets.GITHUB_TOKEN }}
115135
script: |
116136
const now = new Date();
117137
// Date format used: YYYY/MM/DD HH:MM , UTC time(ex: 2023/01/13 08:38)

0 commit comments

Comments
 (0)