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 unnecessary Symbol stringification #8310

Merged
merged 1 commit into from
Jan 19, 2022

Conversation

camelid
Copy link
Member

@camelid camelid commented Jan 19, 2022

This should (slightly) improve performance and enhance code quality.

changelog: none

@rust-highfive
Copy link

r? @Manishearth

(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 Jan 19, 2022
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jan 19, 2022

📌 Commit 7c844ef has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Jan 19, 2022

⌛ Testing commit 7c844ef with merge 92a57f5...

bors added a commit that referenced this pull request Jan 19, 2022
Remove unnecessary `Symbol` stringification

This should (slightly) improve performance and enhance code quality.

changelog: none
@bors
Copy link
Contributor

bors commented Jan 19, 2022

💔 Test failed - checks-action_test

This should (slightly) improve performance and enhance code quality.
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jan 19, 2022

📌 Commit d9f401a has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Jan 19, 2022

⌛ Testing commit d9f401a with merge 16032c5...

@bors
Copy link
Contributor

bors commented Jan 19, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 16032c5 to master...

@bors bors merged commit 16032c5 into rust-lang:master Jan 19, 2022
@bors bors mentioned this pull request Jan 19, 2022
@camelid camelid deleted the rm-stringification branch January 19, 2022 17:12
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