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

Updated known problems section for pre-expansion lints about level attributes #8040

Merged

Conversation

xFrednet
Copy link
Member

Our last three pre-macro expansion lints aren't affected by lint level attributes. This adds a comment to the know problems section of them. I've also updated some CSS to add some spacing after lists on Clippy's lint list:

Before:

image

After:

image


changelog: [print_stdout], [print_stderr], [dbg_macro]: Document how the lint level can be changed and reference #6610

cc: #6610

@rust-highfive
Copy link

r? @camsteffen

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 28, 2021
clippy_lints/src/dbg_macro.rs Outdated Show resolved Hide resolved
util/gh-pages/index.html Outdated Show resolved Hide resolved
@xFrednet xFrednet force-pushed the 6610-known-problems-section-for-pre-expansion branch from 4b2bbb3 to b5b4017 Compare December 3, 2021 18:53
@xFrednet xFrednet force-pushed the 6610-known-problems-section-for-pre-expansion branch from b5b4017 to 17b7478 Compare December 3, 2021 19:24
@camsteffen
Copy link
Contributor

Thanks! @bors r+

@bors
Copy link
Contributor

bors commented Dec 3, 2021

📌 Commit 17b7478 has been approved by camsteffen

@bors
Copy link
Contributor

bors commented Dec 3, 2021

⌛ Testing commit 17b7478 with merge 3cd151d...

@bors
Copy link
Contributor

bors commented Dec 3, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: camsteffen
Pushing 3cd151d to master...

@bors bors merged commit 3cd151d into rust-lang:master Dec 3, 2021
@xFrednet xFrednet deleted the 6610-known-problems-section-for-pre-expansion branch July 9, 2024 20:05
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.

5 participants