fix: rebuild heading-maps (v0.12.5 MyST role fix)#13
Closed
Conversation
Strips MyST inline roles ({index}, etc.) from heading-map keys/values.
Regenerated using: npx translate headingmap
There was a problem hiding this comment.
Pull request overview
Rebuilds the heading-map metadata for the translated “Python Advanced Features” lecture using action-translation v0.12.5, aligning heading-map keys with clean display text (avoiding MyST role pollution).
Changes:
- Removes the top-level
More Language Featuresentry from theheading-mapinpython_advanced_features.md. - Updates the translation state to reflect
tool-version: 0.12.5.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
lectures/python_advanced_features.md |
Updates heading-map entries to the newly generated set (drops the top-level mapping entry). |
.translate/state/python_advanced_features.md.yml |
Records the translation tool version bump to 0.12.5. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
4
to
+6
| mode: UPDATE | ||
| section-count: 7 | ||
| tool-version: 0.12.4 | ||
| tool-version: 0.12.5 |
There was a problem hiding this comment.
PR description says only python_advanced_features.md was updated, but this PR also changes .translate/state/python_advanced_features.md.yml (tool-version bump to 0.12.5). Please update the PR description (or drop the state-file change) so the summary matches the actual diff.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebuild heading-maps using
action-translationv0.12.5 which fixes MyST role pollution (PR #50).What changed: Headings with MyST inline roles (e.g.
{index}`Pandas <single: Pandas>`) were previously stored verbatim as heading-map keys. Now the clean display text is used instead.Generated with:
Files updated: 1 (
python_advanced_features.md)