Skip to content

Remove unnecessary Symbol stringification#8310

Merged
bors merged 1 commit intorust-lang:masterfrom
camelid:rm-stringification
Jan 19, 2022
Merged

Remove unnecessary Symbol stringification#8310
bors merged 1 commit intorust-lang:masterfrom
camelid:rm-stringification

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.
@camelid camelid force-pushed the rm-stringification branch from 7c844ef to d9f401a Compare January 19, 2022 01:03
@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

Comments