Skip to content

docs: add a "glossary" section to the guide, automatically link to it#5933

Merged
davidhewitt merged 5 commits intoPyO3:mainfrom
davidhewitt:glossary
Apr 2, 2026
Merged

docs: add a "glossary" section to the guide, automatically link to it#5933
davidhewitt merged 5 commits intoPyO3:mainfrom
davidhewitt:glossary

Conversation

@davidhewitt
Copy link
Copy Markdown
Member

This PR adds a glossary page to the guide, as an attempt to create common definitions of terms which occur frequently in the guide (and are relatively critical to user understanding).

I added a glossary_linker.py mdbook preprocessor which automatically links the first reference of each term on each page to the glossary (including some specially curated terms found in external glossaries such as the Python docs).

Full disclosure: glossary_linker.py was vibe-coded by Claude with some manual adjustment and review by me. There was no way I would get the regular expressions for the "skip" patterns correct without a lot of trial-and-error 😂

Copy link
Copy Markdown
Contributor

@LilyFirefly LilyFirefly left a comment

Choose a reason for hiding this comment

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

The docs fixes look great! I have not reviewed the glossary linker closely.

@davidhewitt
Copy link
Copy Markdown
Member Author

The only remaining link check error I think is https://pyo3.netlify.app/internal/doc/pyo3/index.html, which I think should be fixed on deploy from main by some of the changes in the noxfile.

I'll proceed to merge.

@davidhewitt davidhewitt enabled auto-merge April 1, 2026 20:57
@davidhewitt davidhewitt added this pull request to the merge queue Apr 1, 2026
Merged via the queue into PyO3:main with commit 96c53e3 Apr 2, 2026
44 of 45 checks passed
@davidhewitt davidhewitt deleted the glossary branch April 2, 2026 00:01
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