-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Rework module_inception
#14753
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
Rework module_inception
#14753
Conversation
r? @Manishearth rustbot has assigned @Manishearth. Use |
cfeac47
to
fa4b49b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on vacation soon, will redirect review
name: Symbol, | ||
name_camel: String, | ||
is_public: bool, | ||
in_body_count: u32, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: docs on what this is
r? clippy |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. r=me with the dogfood warning fixed
This comment has been minimized.
This comment has been minimized.
* Don't check for repetition if the previous module crosses a body boundary. * Don't take a snippet of the entire item. * Check each item's visibility once. * Use `is_from_proc_macro` before linting
b62beeb
to
26cf3ee
Compare
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
is_from_proc_macro
before lintingchangelog: none