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

Fix FP in to_string_in_display #5971

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

giraffate
Copy link
Contributor

Don't emit a lint when .to_string() on anything that is not self

Fix #5967

changelog: Fix FP in to_string_in_display when calling .to_string() on anything that is not self

Don't emit a lint when `.to_string()` on anything that is not `self`
@rust-highfive
Copy link

r? @ebroto

(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 Aug 27, 2020
@ebroto
Copy link
Member

ebroto commented Aug 27, 2020

@bors r+

Thanks!

@bors
Copy link
Collaborator

bors commented Aug 27, 2020

📌 Commit 04bff17 has been approved by ebroto

@bors
Copy link
Collaborator

bors commented Aug 27, 2020

⌛ Testing commit 04bff17 with merge 07c5e9e...

@bors
Copy link
Collaborator

bors commented Aug 27, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: ebroto
Pushing 07c5e9e to master...

@bors bors merged commit 07c5e9e into rust-lang:master Aug 27, 2020
@giraffate giraffate deleted the fix_fp_in_to_string_in_display branch August 28, 2020 01:18
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.

FP to_string_in_display: calling .to_string() on anything that is not "self"
4 participants