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 failing uninlined-format-args tests #9548

Closed
wants to merge 1 commit into from

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented Sep 28, 2022

All these tests are currently failing (generating uncompilable code).

The .fixed and .stderr were hacked without the cargo dev bless, so might be incorrect.

changelog: [uninlined_format_args]: fixed several edge case bugs

CC: @Alexendoo

@rust-highfive
Copy link

r? @giraffate

(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 Sep 28, 2022
@giraffate
Copy link
Contributor

r? @Alexendoo

@rust-highfive rust-highfive assigned Alexendoo and unassigned giraffate Sep 28, 2022
@nyurik nyurik force-pushed the fix-uninlined-fmt branch 2 times, most recently from cba4ef5 to f6c13af Compare October 4, 2022 02:09
@bors
Copy link
Contributor

bors commented Oct 4, 2022

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

@bors
Copy link
Contributor

bors commented Oct 5, 2022

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

All these tests are currently failing (generating uncompilable code).

* Implement a simplified `indoc!` proc macro
* added a `pass_through!(...)` macro test

An edge case with format arguments being wrapped in parentesis

The .fixed and .stderr were hacked without the `cargo dev bless`, so might be incorrect.
@nyurik
Copy link
Contributor Author

nyurik commented Oct 5, 2022

these tests are no longer needed due to #9586

@nyurik nyurik closed this Oct 5, 2022
@nyurik nyurik deleted the fix-uninlined-fmt branch October 5, 2022 21:55
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