Skip to content

Conversation

jakubadamw
Copy link
Contributor

Fixes #4296.

changelog: Extend the use_self lint to suggest uses of Self::Variant

Copy link
Member

@flip1995 flip1995 left a comment

Choose a reason for hiding this comment

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

LGTM overall, 2 small style issues I had

One struct required a temporary `#[allow(dead_code)]` annotation due to
a bug in the Rust compiler: rust-lang/rust#63151.
@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jul 31, 2019
@flip1995
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Jul 31, 2019

📌 Commit 2a13e83 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Jul 31, 2019

⌛ Testing commit 2a13e83 with merge 128647d...

bors added a commit that referenced this pull request Jul 31, 2019
Extend the `use_self` lint to suggest uses of `Self::Variant`

Fixes #4296.

changelog: Extend the `use_self` lint to suggest uses of `Self::Variant`
@bors
Copy link
Contributor

bors commented Jul 31, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 128647d to master...

@bors bors merged commit 2a13e83 into rust-lang:master Jul 31, 2019
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.

New lint: Suggest using Self::Variant instead of TheType::Variant
3 participants