-
First Check
Commit to Help
Example Code"
# Development - Contributing
First, you might want to see the basic ways to [help Typer and get help](help-typer.md).
"
Should (perhaps) be changed to:
"
# Development - Contributing
First, you might want to see the basic ways to [help Typer and get help](/docs/help-typer.md).
"DescriptionAt the start of the documentation of Operating SystemWindows Operating System DetailsNo response Project Version0.62.2 Python VersionPython 3.13.7 Additional Contexthttps://github.com/fastapi/typer/blob/master/help-typer.md: The page is rather at docs: |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 2 replies
-
|
this is real, but it's a GitHub overview-tab edge case. The normal
|
Beta Was this translation helpful? Give feedback.
-
|
Right, yes, there's indeed a 404 when clicking the help-typer link from over at https://github.com/fastapi/typer?tab=contributing-ov-file. We'd be open to correct this as long as all other links/use-cases from the docs etc remain valid. |
Beta Was this translation helpful? Give feedback.
-
|
Right - okay, I am back. I looked into this a bit more, it seems /docs/ or any alternative of it (i.e ./docs/(...), etc) makes Github render it correctly, but then MkDocs complains about issues, if I understand correctly because it probably parses it as Now other alternatives make the problem the other way around (basically from Github it gives 404 again because it tries to get to The middle-ground I found was using the live page, i.e : https://typer.tiangolo.com/help-typer/ for it to point to, this gives both no issues from MkDocs and the GitHub renders it correctly to the page. Altough this approach uses the live page - which I am not sure is favorable - but i see it is getting used in a lot of other links regardless... I also tested this: Some useful links I found along the way: Would using the published docs URL be acceptable here? If so, I’m happy to open a PR with that change, unless maintainers prefer to handle it directly. |
Beta Was this translation helpful? Give feedback.


Fix merged in #1826