Skip to content

chore(locales): move French review hashes into locales/.review.json - #57

Open
ValgulNecron wants to merge 1 commit into
mainfrom
chore/move-fr-review-hashes
Open

ValgulNecron wants to merge 1 commit into
mainfrom
chore/move-fr-review-hashes

Conversation

@ValgulNecron

@ValgulNecron ValgulNecron commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

Before: the French fr.json files for minecraft-java, terraria and valheim carry sourceHash inline. When Weblate rewrites a translation file, it drops unknown keys, so its first edit would delete those hashes.

After: the messages are unchanged and the 210 hashes (123, 69 and 18) move unchanged into each module's locales/.review.json. This is the location that gp-module i18n stamp and i18n-check read since Gameplane PR #849 (spec 019 T078).

How: I ran gp-module i18n stamp <module> (built from Gameplane master a536896) on the three modules, then checked with a script that every message, description and hash matches the original file. The CI i18n-check@master job already understands the review file.


Generated by Claude Code

Weblate drops unknown keys such as sourceHash when it rewrites a
translation file. Ran gp-module i18n stamp on minecraft-java, terraria
and valheim: messages are unchanged, the 210 hashes move unchanged to
locales/.review.json (Gameplane spec 019 T078).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Claude <noreply@anthropic.com>
@ValgulNecron ValgulNecron self-assigned this Nov 9, 2026
@coderabbitai

coderabbitai Bot commented Nov 9, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ad5eb05f-ca3d-4178-bdb8-052ceb6cf633
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants