Skip to content

doc: use absolute links in style guide, add link to cookiecutter#5656

Merged
petrasovaa merged 2 commits into
OSGeo:mainfrom
petrasovaa:fix-contributing-guide-links
Jul 15, 2026
Merged

doc: use absolute links in style guide, add link to cookiecutter#5656
petrasovaa merged 2 commits into
OSGeo:mainfrom
petrasovaa:fix-contributing-guide-links

Conversation

@petrasovaa

Copy link
Copy Markdown
Contributor

Relative links to files in repo don't work anymore now when style guide is part of documentation.

Also adds a link to the addon template in development documentation.

@echoix echoix 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.

Can't it be properly configured on the tool used side? It seems like a regression for us to have this PR

@github-actions github-actions Bot added docs markdown Related to markdown, markdown files labels May 15, 2025
@echoix

echoix commented May 21, 2025

Copy link
Copy Markdown
Member

To see if setting docs_dir to a relative path like "../../" (appropriate for the location of the root of the repo with respect to the config file location).

https://www.mkdocs.org/user-guide/configuration/#docs_dir

Also consider if there's blockers to have the mkdocs.yml config file at the root, to be able to build with "mkdocs build" without changing any directories first.

@petrasovaa
petrasovaa enabled auto-merge (squash) July 15, 2026 15:10
@petrasovaa
petrasovaa merged commit c6459dc into OSGeo:main Jul 15, 2026
19 checks passed
@petrasovaa
petrasovaa deleted the fix-contributing-guide-links branch July 15, 2026 20:07
@github-actions github-actions Bot added this to the 8.6.0 milestone Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs markdown Related to markdown, markdown files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants