Skip to content

Doc: Add glossary - #322

Merged
loichuder merged 1 commit into
mainfrom
glossary
Jul 27, 2026
Merged

Doc: Add glossary#322
loichuder merged 1 commit into
mainfrom
glossary

Conversation

@loichuder

@loichuder loichuder commented Jul 6, 2026

Copy link
Copy Markdown
Member

Reimplemention of Andy's initiative during the ewoks doc week

image

Requires to add :term to reference glossary words which is a bit painful (done by PO with Codex). Not sure we will be able to do it consistently.

Comment thread doc/howtoguides/job.rst Outdated
Comment thread doc/reference/glossary.rst Outdated
Comment thread doc/tutorials/create_workflow.rst Outdated
Comment thread doc/reference/glossary.rst Outdated
Comment thread doc/tutorials/hello_world.rst Outdated
Comment thread doc/index.rst Outdated
@loichuder
loichuder requested a review from a team July 27, 2026 09:30
@loichuder
loichuder marked this pull request as ready for review July 27, 2026 09:31

@poautran poautran left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Other than these 2 comments LGTM! Minor thing: there are 3 dictionnary to be replaced by dictionary. Also the CI needs fixing I guess.

Comment thread doc/reference/glossary.rst Outdated
Comment thread doc/reference/glossary.rst Outdated
@loichuder

Copy link
Copy Markdown
Member Author

Thanks for the review! The CI failure comes from horejsek/python-fastjsonschema#211

@loichuder
loichuder merged commit b397e7a into main Jul 27, 2026
9 of 12 checks passed
@loichuder

Copy link
Copy Markdown
Member Author

Oops, missed a few typos. Fixed in dc70caa

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