Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC: Use MyST syntax for linking other files #3499

Merged
merged 2 commits into from
Oct 10, 2024
Merged

DOC: Use MyST syntax for linking other files #3499

merged 2 commits into from
Oct 10, 2024

Conversation

seisman
Copy link
Member

@seisman seisman commented Oct 9, 2024

Description of proposed changes

To link to a file, currently we use syntax like:

{doc}`/techref/encodings`

MyST also allows using Markdown link syntax:

[](/techref/encodings)

which is IMHO more readable and more familiar to Markdown users.

This PR updates to the MyST syntax.

Reference: https://myst-parser.readthedocs.io/en/latest/syntax/cross-referencing.html#inline-links-with-implicit-text

@seisman seisman added documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog needs review This PR has higher priority and needs review. labels Oct 9, 2024
@seisman seisman added this to the 0.14.0 milestone Oct 9, 2024
@michaelgrund michaelgrund added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Oct 9, 2024
Co-authored-by: Yvonne Fröhlich <[email protected]>
@seisman seisman merged commit c471769 into main Oct 10, 2024
10 checks passed
@seisman seisman deleted the doc/doclink branch October 10, 2024 00:16
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants