ci(translation-sync): remove redundant push trigger

[lizhengfeng101]

Description

The translation-sync workflow (.github/workflows/translation-sync.yml) currently triggers on both push and pull_request events. Since all code merges into main via pull requests, the pull_request trigger already validates translation sync before merge — running the same check again on the resulting push to main is redundant and wastes CI resources.

Current trigger configuration (lines 9–19):

on:
  push:
    paths:
      - "README*.md"
      - "pages/src/content/docs/**"
      - "scripts/github-actions/check-translation-sync.*"
  pull_request:
    paths:
      - "README*.md"
      - "pages/src/content/docs/**"
      - "scripts/github-actions/check-translation-sync.*"

Scope

• File: .github/workflows/translation-sync.yml
• Area: CI workflow triggers
• Remove the push: trigger block (lines 10–14), keeping only pull_request:

Acceptance Criteria

• The on: section only contains a pull_request trigger with the existing paths: filter
• The workflow no longer runs on push to main
• All other workflow behavior (steps, container, env vars) remains unchanged

Context

Introduced in PR #455. The workflow validates README heading structure (blocking) and docs translation coverage (non-blocking). Since the non-blocking docs check uses github.event.pull_request.base.sha / head.sha, it is inherently PR-only anyway — the push trigger only exercises the blocking README check redundantly after merge.

[Shaurya2k06]

Happy to take this up!

cc @lizhengfeng101

[tnknick14]

I can handle this. The fix is straightforward — remove the push block from the on: section, keeping only pull_request with its existing paths filter.

One question: should I leave the pull_request paths filter unchanged, or do you want to review whether those patterns still match the intended files? The current filter targets README*.md, pages/src/content/docs/**, and the sync script — if any of those paths have shifted, this is a good time to correct them.

I'll submit a PR with the change.

[lizhengfeng101]

Thanks for jumping on this, @tnknick14 — really appreciate the initiative, and great question about the paths filter.

Two things:

1. This one's already claimed. @Shaurya2k06 offered to take it a bit earlier (08:34) and has already opened ci(translation-sync): remove redundant push trigger #460 with the fix, so let's let that one run its course to avoid duplicate work. Sorry for the overlap!

2. On your paths question — the filter should stay unchanged. I double-checked and all three patterns still match real files:

   • README*.md → README.md + the 4 localized versions (zh-CN, ja-JP, ko-KR, ru-RU) ✅
   • pages/src/content/docs/** → directory exists ✅
   • scripts/github-actions/check-translation-sync.* → .js + .test.js ✅

   So no path corrections are needed here — the scope is purely removing the push: block.

Would love to have you contribute though — there are several other good first issue tickets open, feel free to grab one and I'll help you get it merged. 🙏