fix: rebuild heading-maps (v0.12.5 MyST role fix)#82
Conversation
Strips MyST inline roles ({index}, etc.) from heading-map keys/values.
Regenerated using: npx translate headingmap
✅ Deploy Preview for majestic-griffin-10b166 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Rebuilds heading-map metadata using action-translation v0.12.5 so heading-map keys use cleaned display text (avoiding MyST role “pollution”) and records the updated tool version in translation state.
Changes:
- Removes the now-unneeded H1 heading-map entry for “More Language Features” in
python_advanced_features.md. - Updates the translation state file 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 frontmatter heading-map output from the regenerated heading-map. |
| .translate/state/python_advanced_features.md.yml | Records the heading-map generation tool version bump to 0.12.5. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 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 "Files updated: 1 (python_advanced_features.md)" but this PR also changes .translate/state/python_advanced_features.md.yml (tool-version bump). Please update the description (or clarify it only counts translated lecture files) to match the actual diff.
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)