Skip to content

Update rustdoc passes documentation#2827

Open
cijiugechu wants to merge 1 commit into
rust-lang:mainfrom
cijiugechu:docs/rustdoc-passes-update
Open

Update rustdoc passes documentation#2827
cijiugechu wants to merge 1 commit into
rust-lang:mainfrom
cijiugechu:docs/rustdoc-passes-update

Conversation

@cijiugechu
Copy link
Copy Markdown
Member

Align the rustdoc internals guide with the current pass definitions and execution order.

Align the rustdoc internals guide with the current pass definitions and execution order so the documented names, conditions, and coverage behavior match the source.
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 8, 2026

Thanks for the PR. If you have write access, feel free to merge this PR if it does not need reviews. You can request a review using r? rustc-dev-guide or r? <username>.

@rustbot rustbot added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Apr 8, 2026
@cijiugechu
Copy link
Copy Markdown
Member Author

r? rustc-dev-guide

Copy link
Copy Markdown
Member

@jyn514 jyn514 left a comment

Choose a reason for hiding this comment

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

This has a lot of structural changes and no rationale. Can you please explain why you made this change and how you think it improves the docs?

View changes since this review

Comment thread src/rustdoc-internals.md
Comment on lines +96 to +100
the cleaned [`AST`][ast]. Several of these passes are `lint`s and reports, but
some of them mutate or generate new items. These are implemented in the
[`librustdoc/passes`] directory, generally one module per pass. Unlike the
previous set of [`AST`][ast] transformations, these passes are run on the
cleaned crate.
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.

nit: we try to use one sentence per line in new docs.

@rustbot rustbot added S-waiting-on-author Status: this PR is waiting for additional action by the OP and removed S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content labels May 21, 2026
@cijiugechu
Copy link
Copy Markdown
Member Author

This has a lot of structural changes and no rationale. Can you please explain why you made this change and how you think it improves the docs?

View changes since this review

Ah, my bad. I overlooked whether this restructure improved docs. I was focused on reflecting the current source code's structure.

@cijiugechu
Copy link
Copy Markdown
Member Author

cijiugechu commented May 23, 2026

I’ll try to revisit this these days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: this PR is waiting for additional action by the OP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants