Skip to content

docs(CLAUDE.md): drop stale translation_table.yaml key-file reference#605

Merged
realmarcin merged 1 commit into
masterfrom
docs/fix-stale-translation-table-ref
Jul 21, 2026
Merged

docs(CLAUDE.md): drop stale translation_table.yaml key-file reference#605
realmarcin merged 1 commit into
masterfrom
docs/fix-stale-translation-table-ref

Conversation

@realmarcin

Copy link
Copy Markdown
Collaborator

One-line doc fix: CLAUDE.md listed kg_microbe/transform_utils/translation_table.yaml as a key file, but it's been replaced by in-code str.maketrans constants (TRANSLATION_TABLE_FOR_IDS/TRANSLATION_TABLE_FOR_LABELS in constants.py) — nothing loads the YAML. Folded the entity-translation note into the constants.py entry. Surfaced during branch-cleanup analysis.

🤖 Generated with Claude Code

The file was replaced by in-code str.maketrans tables
(TRANSLATION_TABLE_FOR_IDS / TRANSLATION_TABLE_FOR_LABELS in constants.py);
nothing loads the YAML anymore. Fold the entity-type-translation note into
the constants.py entry.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 21, 2026 19:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates repository guidance in CLAUDE.md to remove a stale reference to kg_microbe/transform_utils/translation_table.yaml and instead document the in-code translation tables now used by transforms (TRANSLATION_TABLE_FOR_IDS / TRANSLATION_TABLE_FOR_LABELS via str.maketrans).

Changes:

  • Removes the obsolete translation_table.yaml “Key Files” entry.
  • Expands the constants.py “Key Files” entry to note the translation-table constants and how they’re applied.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@realmarcin
realmarcin merged commit 77ccabf into master Jul 21, 2026
4 checks passed
@realmarcin
realmarcin deleted the docs/fix-stale-translation-table-ref branch July 21, 2026 20:11
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