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

Add a temporary workaround for multiline formart arg inlining #9599

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented Oct 6, 2022

per suggestion in
rust-lang/rust#102729 (comment)

workaround for an internal crash when handling multi-line format argument inlining.

changelog: none

(no point for changelog because it is still a new lint being introduced)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 6, 2022
per suggestion in
rust-lang/rust#102729 (comment)

workaround for an internal crash when handling multi-line format
argument inlining.
@flip1995
Copy link
Member

flip1995 commented Oct 6, 2022

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Oct 6, 2022

📌 Commit cfd6c8d has been approved by flip1995

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 6, 2022

⌛ Testing commit cfd6c8d with merge 6dd0c3a...

@bors
Copy link
Contributor

bors commented Oct 6, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 6dd0c3a to master...

@bors bors merged commit 6dd0c3a into rust-lang:master Oct 6, 2022
@nyurik nyurik deleted the inline-crash branch October 6, 2022 14:26
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.

5 participants