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

Remove to_string()s from CompilerLintFunctions #4177

Merged
merged 1 commit into from Jun 6, 2019
Merged

Remove to_string()s from CompilerLintFunctions #4177

merged 1 commit into from Jun 6, 2019

Conversation

ghost
Copy link

@ghost ghost commented Jun 6, 2019

changelog: none

@phansch
Copy link
Member

phansch commented Jun 6, 2019

@bors r+ thanks!

@bors
Copy link
Contributor

bors commented Jun 6, 2019

📌 Commit b36d7a0 has been approved by phansch

@bors
Copy link
Contributor

bors commented Jun 6, 2019

⌛ Testing commit b36d7a0 with merge e616800...

bors added a commit that referenced this pull request Jun 6, 2019
… r=phansch

Remove `to_string()`s from CompilerLintFunctions

changelog: none
@bors
Copy link
Contributor

bors commented Jun 6, 2019

💔 Test failed - status-appveyor

@flip1995
Copy link
Member

flip1995 commented Jun 6, 2019

After #4178 is merged, this should work.

@bors retry

@bors
Copy link
Contributor

bors commented Jun 6, 2019

⌛ Testing commit b36d7a0 with merge f65308c...

bors added a commit that referenced this pull request Jun 6, 2019
… r=phansch

Remove `to_string()`s from CompilerLintFunctions

changelog: none
@bors
Copy link
Contributor

bors commented Jun 6, 2019

💔 Test failed - checks-travis

@flip1995
Copy link
Member

flip1995 commented Jun 6, 2019

@bors retry (dogfood shenanigans on Mac)

@bors
Copy link
Contributor

bors commented Jun 6, 2019

⌛ Testing commit b36d7a0 with merge 71be6f6...

bors added a commit that referenced this pull request Jun 6, 2019
… r=phansch

Remove `to_string()`s from CompilerLintFunctions

changelog: none
@bors
Copy link
Contributor

bors commented Jun 6, 2019

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

@bors bors merged commit b36d7a0 into rust-lang:master Jun 6, 2019
@ghost ghost deleted the compiler_lint_functions_fewer_strings branch June 7, 2019 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants