Skip to content

Conversation

psychedelicious
Copy link
Collaborator

Summary

  • Port clean translations script to node. It checks for unused translation strings that exist in en.json but are not referenced in codebase. Unfortunately going the other way round (finding missing translations) is very hard.
  • Add it to frontend CI checks.

Related Issues / Discussions

n/a

QA Instructions

n/a

Merge Plan

n/a

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@github-actions github-actions bot added CI-CD Continuous integration / Continuous delivery frontend-deps PRs that change frontend dependencies frontend PRs that change frontend files labels Sep 4, 2025
@psychedelicious
Copy link
Collaborator Author

I let the new CI check run twice on the PR, before and after committing the cleaned translations file.

So looks like it is working correctly

@psychedelicious psychedelicious enabled auto-merge (rebase) September 5, 2025 00:58
@psychedelicious psychedelicious merged commit 8a81c05 into main Sep 5, 2025
13 checks passed
@psychedelicious psychedelicious deleted the psyche/tidy/ui/remove-unused-translations branch September 5, 2025 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-CD Continuous integration / Continuous delivery frontend PRs that change frontend files frontend-deps PRs that change frontend dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants