Skip to content

Conversation

grant0417
Copy link
Contributor

@grant0417 grant0417 commented Dec 12, 2024

Fixes #5244

This PR fixes the issue in #6416 (duplicate of #5244), I have added both the before and after case of the formatting in tests/.

Edit (ytmimi): I updated the description

@ytmimi
Copy link
Contributor

ytmimi commented Dec 12, 2024

Thanks for the PR! Can you add the input from #5244 as a test case as well?

@grant0417
Copy link
Contributor Author

Yep, changed the tests to those.

@ytmimi
Copy link
Contributor

ytmimi commented Jan 21, 2025

Ran our Diff-Check and things look good

@ytmimi ytmimi merged commit 6f7aeed into rust-lang:master Jan 21, 2025
26 checks passed
@ytmimi ytmimi added release-notes Needs an associated changelog entry and removed pr-not-reviewed labels Jan 21, 2025
@ytmimi
Copy link
Contributor

ytmimi commented Jan 21, 2025

Thanks for the help on this, and congrats on your first contribution to rustfmt 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes Needs an associated changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

wrap_comments does not ensure code blocks start on a new line

3 participants