Skip to content

Add --type flag to dev new_lint #9182

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 4 commits into from
Jul 27, 2022
Merged

Conversation

Serial-ATA
Copy link
Contributor

changelog: none

This only works with directories, --type={copies, doc, ...} won't work. Should they be counted?

@rust-highfive
Copy link

r? @xFrednet

(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 Jul 15, 2022
@Serial-ATA Serial-ATA force-pushed the new_lint-type-flag branch from e772081 to dde41b5 Compare July 15, 2022 17:39
Copy link
Member

@xFrednet xFrednet left a comment

Choose a reason for hiding this comment

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

Hey, this PR has not been forgotten, my two weeks have just been pretty full, with rl stuff. I'm away over the weekend, but should be able to give you a review next week. I've added one comment about something that stuck out to me :)

@bors
Copy link
Contributor

bors commented Jul 25, 2022

☔ The latest upstream changes (presumably #9130) made this pull request unmergeable. Please resolve the merge conflicts.

@Serial-ATA Serial-ATA force-pushed the new_lint-type-flag branch from dde41b5 to 51cd5a8 Compare July 26, 2022 02:35
Copy link
Member

@xFrednet xFrednet left a comment

Choose a reason for hiding this comment

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

The implementation looks good overall, I found some small nits, where some might be too nitpicky. 😅

Could you maybe also be so kind to update the related docs in book/src/development/adding_lints.md? Mentioning the type option is probably enough 🙃

@xFrednet
Copy link
Member

xFrednet commented Jul 27, 2022

I suggest rephrasing one paragraph in the documentation. Otherwise, everything looks good to me. If you agree with the change, you can commit it and merge it with (at)bors r=xFrednet

Thank you for this addition, it was something I wanted to see for a longer time!

@bors delegate+

@bors
Copy link
Contributor

bors commented Jul 27, 2022

✌️ @Serial-ATA can now approve this pull request

@Serial-ATA Serial-ATA force-pushed the new_lint-type-flag branch from 179b81f to 8319196 Compare July 27, 2022 22:00
@Serial-ATA
Copy link
Contributor Author

@bors r=xFrednet

@bors
Copy link
Contributor

bors commented Jul 27, 2022

📌 Commit 8319196 has been approved by xFrednet

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jul 27, 2022

⌛ Testing commit 8319196 with merge cdd4dfc...

@bors
Copy link
Contributor

bors commented Jul 27, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing cdd4dfc to master...

@bors bors merged commit cdd4dfc into rust-lang:master Jul 27, 2022
@bors bors mentioned this pull request Jul 27, 2022
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.

4 participants