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

trigger string_lit_as_bytes when literal has escapes #4808

Merged
merged 1 commit into from
Nov 28, 2019

Conversation

euclio
Copy link
Contributor

@euclio euclio commented Nov 12, 2019


changelog: fix string_lit_as_bytes false negative

Depends on rust-lang/rust#66349.

Fixes #4796.

@flip1995 flip1995 added the S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work label Nov 13, 2019
@euclio euclio force-pushed the string-lit-as-bytes branch from 28eeae7 to bad1624 Compare November 17, 2019 02:29
@euclio
Copy link
Contributor Author

euclio commented Nov 17, 2019

@flip1995 This is now unblocked.

Copy link
Member

@phansch phansch left a comment

Choose a reason for hiding this comment

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

thanks!
r=me once Clippy builds again

@phansch phansch removed the S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work label Nov 18, 2019
@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 18, 2019
@flip1995
Copy link
Member

flip1995 commented Nov 23, 2019

@bors r=phansch rollup

1 similar comment
@flip1995
Copy link
Member

@bors r=phansch rollup

@flip1995
Copy link
Member

bors lost this PR. Once the pile from the rustup is merged, we have to resync the bors queue.

@flip1995
Copy link
Member

ping @euclio. This PR only needs a rebase.

@euclio euclio force-pushed the string-lit-as-bytes branch from bad1624 to d33ad45 Compare November 26, 2019 22:07
@euclio
Copy link
Contributor Author

euclio commented Nov 26, 2019

Rebased, but it won't work until the rustup goes through.

@flip1995 flip1995 added S-waiting-on-bors Status: The marked PR was approved and is only waiting bors and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Nov 27, 2019
@phansch
Copy link
Member

phansch commented Nov 28, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Nov 28, 2019

📌 Commit d33ad45 has been approved by phansch

@bors
Copy link
Contributor

bors commented Nov 28, 2019

⌛ Testing commit d33ad45 with merge d6accfc...

bors added a commit that referenced this pull request Nov 28, 2019
trigger string_lit_as_bytes when literal has escapes

---

changelog: fix string_lit_as_bytes false negative

Depends on rust-lang/rust#66349.

Fixes #4796.
@bors
Copy link
Contributor

bors commented Nov 28, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: phansch
Pushing d6accfc to master...

@bors bors merged commit d33ad45 into rust-lang:master Nov 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: The marked PR was approved and is only waiting bors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

string_lit_as_bytes lint isn't trigged when string contains escaped characters
4 participants