Skip to content

Conversation

@betelgeuse
Copy link

For people new to clippy, it's better to be explicit. The terminology of allow-by-default could also mean that the lint is enabled (vs. being allowed in the source code).


Please write a short comment explaining your change (or "none" for internal only changes)

changelog: Document levels explicitly in the book

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 30, 2025
@rustbot
Copy link
Collaborator

rustbot commented Dec 30, 2025

r? @flip1995

rustbot has assigned @flip1995.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

For people new to clippy, it's better to be explicit.
The terminology of allow-by-default could also mean
that the lint is enabled (vs. being allowed in the source code).
Comment on lines +26 to +29
The lint levels are:
* allow: can appear in the source code
* warn: emits a warning
* deny: can't appear in the source code
Copy link
Member

Choose a reason for hiding this comment

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

The rustc book already explains all lint levels: https://doc.rust-lang.org/rustc/lints/levels.html

I suggest to just link to that from here.

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants