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.

Sign up for GitHub

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

Jump to bottom

internal: use string interpolation in more places #17315

Merged
merged 2 commits into from
Jun 3, 2024
Merged

internal: use string interpolation in more places #17315

merged 2 commits into from
Jun 3, 2024

Conversation

hamirmahal
Copy link
Contributor

No description provided.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 30, 2024
hamirmahal
hamirmahal commented May 30, 2024
View reviewed changes
Copy link
Contributor Author

@hamirmahal hamirmahal left a comment •
edited
Loading

Choose a reason for hiding this comment

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

This pull request shouldn't cause any external changes. Its goal is just to improve readability.

@Veykril
Copy link
Member

Veykril commented Jun 3, 2024

I sure hope you automated this somehow 😬 Thanks!
@bors r+

@bors
Copy link
Collaborator

bors commented Jun 3, 2024

📌 Commit 7fa555f has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jun 3, 2024

⌛ Testing commit 7fa555f with merge cb6b808...

@bors
Copy link
Collaborator

bors commented Jun 3, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing cb6b808 to master...

@bors bors merged commit cb6b808 into rust-lang:master Jun 3, 2024
11 checks passed
@hamirmahal hamirmahal deleted the style/simplify-string-interpolation branch June 3, 2024 19:52
@hamirmahal
Copy link
Contributor Author

You're welcome!

@lnicola lnicola changed the title style: simplify string interpolation internal: use string interpolation in more places Jun 4, 2024
This was referenced Jun 11, 2024
Merged
Merged
@philips-software-forest-releaser philips-software-forest-releaser bot mentioned this pull request Jun 18, 2024
Merged
@philips-software-forest-releaser philips-software-forest-releaser bot mentioned this pull request Jun 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@hamirmahal @Veykril @bors @rustbot