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

Fix a few documentation/spelling nits #1477

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

tchebb
Copy link
Contributor

@tchebb tchebb commented Mar 16, 2025

See individual commit messages for details. If you'd each commit in a separate PR I can do that, but I figure merging this without squashing is easier given how small each fix is.

tchebb added 3 commits March 16, 2025 14:23
According to "cargo doc --help", --all is a deprecated alias for
--workspace. Fix that, update the test name to one that hasn't been
deleted, and use --exact to only run the specified one.
The multiple re-exports of the same item confuses rustdoc, and currently
other items that try to link to subclass actually link to the deprecated
is_subclass. I reported this bug to rustdoc[1], and a maintainer
suggested swapping the re-export order as a workaround.

[1] rust-lang/rust#136777
We were losing the brackets around attribute names, since rustdoc saw
them as links. Fix that up, and also update a couple links that still
pointed to the deprecated is_subclass attribute.
@tchebb tchebb changed the title Fix a few documentation nits Fix a few documentation/spelling nits Mar 17, 2025
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.

1 participant