Skip to content

docs: Replace relative URL by Rust item links#477

Merged
iliekturtles merged 1 commit into
iliekturtles:masterfrom
caspermeijn:docs
Oct 22, 2024
Merged

docs: Replace relative URL by Rust item links#477
iliekturtles merged 1 commit into
iliekturtles:masterfrom
caspermeijn:docs

Conversation

@caspermeijn

Copy link
Copy Markdown
Contributor

Some links in the documenation are broken. cargo doc will resolve item names when used in markdown links. The tool will report unresolved links as warning so that they can be fixed. Replace any relative URL by such item link. See https://doc.rust-lang.org/rustdoc/write-documentation/linking-to-items-by-name.html

@iliekturtles iliekturtles left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks so much for the PR and sorry for the delays in review. I have one question about some Self links and I think everything else looks good.

Comment thread src/quantity.rs Outdated
Some links in the documenation are broken. `cargo doc` will resolve item names when used in markdown links. The tool will report unresolved links as warning so that they can be fixed. Replace any relative URL by such item link. See https://doc.rust-lang.org/rustdoc/write-documentation/linking-to-items-by-name.html
@iliekturtles iliekturtles merged commit 6ea19bd into iliekturtles:master Oct 22, 2024
@iliekturtles

Copy link
Copy Markdown
Owner

Thanks for the PR! Finally merged!

@caspermeijn caspermeijn deleted the docs branch May 9, 2025 06:57
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