Skip to content

I18n: Add i18n-audit.mjs to Github Actions CI and fix remaining issues#1640

Merged
benjaminshafii merged 2 commits intodifferent-ai:devfrom
johnnyshields:i18n-add-to-ci
May 4, 2026
Merged

I18n: Add i18n-audit.mjs to Github Actions CI and fix remaining issues#1640
benjaminshafii merged 2 commits intodifferent-ai:devfrom
johnnyshields:i18n-add-to-ci

Conversation

@johnnyshields
Copy link
Copy Markdown
Contributor

@johnnyshields johnnyshields commented May 4, 2026

This PR includes the changes from #1639.

This PR:

  • Adds scripts/i18n-audit.mjs --ci to run in the Github Actions CI
  • Does 2 small fixes so CI passes:
    • Adds 4 missing translations (missing from EN and all other locales)
    • Prune unused translation keys

This audit script is lightweight and aimed at catching mistakes.

In order to make merging future PRs easy, the script intentionally does not NOT fail the CI in these cases:

  1. If translations are missing from some locales (as the app will automatically fallback to English)
  2. If there are unused/stale keys (these can be removed at a later time using scripts/i18n-audit.mjs --prune
  3. Just for now, the "missing" {plural} placeholders don't cause failures; these will be removed in the next PR.

(You will still see the above cases reported as issues in the CI output log, but it doesn't cause the run to fail)

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 May 4, 2026 9:06pm

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 4, 2026

@johnnyshields is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

- Prune unused I18n keys
- Add 4 missing translations to English only
- Audit script should always print unused keys
@benjaminshafii benjaminshafii merged commit 32db283 into different-ai:dev May 4, 2026
9 of 14 checks passed
@johnnyshields johnnyshields deleted the i18n-add-to-ci branch May 5, 2026 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants