Let empty files be empty files (reprise) - #6940
Conversation
|
@notpeter can you remind me why we're repurposing the Wouldn't it make more sense to create some new distinct test cases for what we're working on. |
|
When I was doing that PR, I was really bummed when it felt appropriate to duplicate the tree. In this PR, I've now switched from duplicated versions and instead added |
There was a problem hiding this comment.
can we avoid making any changes to the reorder_modules test cases. It's still not clear to me why we're updating these existing test cases. I think we should create new distinct test cases that don't modify any of the old tests.
|
@rustbot author |
|
Reminder, once the PR becomes ready for a review, use |
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
@rustbot ready |
Starting with Style Edition 2027, rustfmt leaves an empty formatted file at zero bytes instead of adding a final newline.
Style Edition 2024 and earlier retain the existing behavior of formatting an empty file as a single newline.