Skip to content

Remove hidden code lines in Clippy's lint list #7887

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

Merged
merged 1 commit into from
Oct 26, 2021

Conversation

xFrednet
Copy link
Member

This PR removes code lines from Clippy's lint list, which would also be hidden, when generating docs with rustdoc.

"A picture is worth a thousand words"... and here are even two pictures:

Before:

image

After:

image


changelog: none

r? @camsteffen (Since you implemented the code block filtering 🙃 )

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 26, 2021
@xFrednet xFrednet force-pushed the 7172-hiding-hidden-lines-online branch from 7f793aa to dea9402 Compare October 26, 2021 20:33
@flip1995
Copy link
Member

flip1995 commented Oct 26, 2021

(Since you implemented the code block filtering 🙃 )

And I implemented this (ignoring "# ") last time in the python doc gen script. 😄

@bors r+ Thanks!

@bors
Copy link
Contributor

bors commented Oct 26, 2021

📌 Commit dea9402 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Oct 26, 2021

⌛ Testing commit dea9402 with merge 0d07ec1...

@xFrednet
Copy link
Member Author

The old python script ^^. I actually didn't know that the script supported this. Nice 👍

@bors
Copy link
Contributor

bors commented Oct 26, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 0d07ec1 to master...

@bors bors merged commit 0d07ec1 into rust-lang:master Oct 26, 2021
@xFrednet
Copy link
Member Author

I've scanned through the gh-pages commit 48e7fe7 and everything is looking fine 👍

@xFrednet xFrednet deleted the 7172-hiding-hidden-lines-online branch October 26, 2021 21:20
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