Skip to content

Conversation

TaKO8Ki
Copy link
Member

@TaKO8Ki TaKO8Ki commented Apr 10, 2022

No description provided.

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Apr 10, 2022
@rust-highfive
Copy link
Contributor

r? @matthewjasper

(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 Apr 10, 2022
@compiler-errors
Copy link
Member

hm, @TaKO8Ki why did you change just this one and not other similar usages? anyways, I guess it's fine.

r? @compiler-errors
@bors r+ rollup=always

@bors
Copy link
Collaborator

bors commented Apr 10, 2022

📌 Commit 80152ed has been approved by compiler-errors

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 10, 2022
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Apr 10, 2022
…rmat, r=compiler-errors

Use `to_string` instead of `format!`
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Apr 11, 2022
…rmat, r=compiler-errors

Use `to_string` instead of `format!`
@TaKO8Ki TaKO8Ki force-pushed the use-to-string-instead-of-format branch 2 times, most recently from f968581 to fbc1f9f Compare April 11, 2022 01:42
@TaKO8Ki TaKO8Ki requested a review from compiler-errors April 11, 2022 01:45
@TaKO8Ki
Copy link
Member Author

TaKO8Ki commented Apr 11, 2022

@rustbot label -S-waiting-on-bors +S-waiting-on-review

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Apr 11, 2022
@TaKO8Ki
Copy link
Member Author

TaKO8Ki commented Apr 11, 2022

@compiler-errors I replaced rest of format!("{}".. with to_string in compiler/.

@TaKO8Ki TaKO8Ki requested a review from compiler-errors April 11, 2022 07:10
@bors
Copy link
Collaborator

bors commented Apr 11, 2022

☔ The latest upstream changes (presumably #95931) made this pull request unmergeable. Please resolve the merge conflicts.

@TaKO8Ki TaKO8Ki force-pushed the use-to-string-instead-of-format branch from 6e00d4d to 29c4128 Compare April 11, 2022 22:51
@compiler-errors
Copy link
Member

@bors r+ rollup=always

please open another PR if you want to change the other usages of format!("{}", ..) instead of pushing to this branch after it's been approved. thanks!

@bors
Copy link
Collaborator

bors commented Apr 12, 2022

📌 Commit 29c4128 has been approved by compiler-errors

@bors bors removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 12, 2022
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 12, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 12, 2022
…askrgr

Rollup of 7 pull requests

Successful merges:

 - rust-lang#95320 (Document the current MIR semantics that are clear from existing code)
 - rust-lang#95722 (pre-push.sh: Use python3 if python is not found)
 - rust-lang#95881 (Use `to_string` instead of `format!`)
 - rust-lang#95909 (rustdoc: Reduce allocations in a `theme` function)
 - rust-lang#95910 (Fix crate_type attribute to not warn on duplicates)
 - rust-lang#95920 (use `Span::find_ancestor_inside` to get right span in CastCheck)
 - rust-lang#95936 (Fix a bad error message for `relative paths are not supported in visibilities` error)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 3ff5cb2 into rust-lang:master Apr 12, 2022
@rustbot rustbot added this to the 1.62.0 milestone Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants