Skip to content

Format an empty file with 0 lines instead of 1 line#6567

Closed
tyilo wants to merge 4 commits intorust-lang:mainfrom
tyilo:empty-file-0-lines
Closed

Format an empty file with 0 lines instead of 1 line#6567
tyilo wants to merge 4 commits intorust-lang:mainfrom
tyilo:empty-file-0-lines

Conversation

@tyilo
Copy link
Copy Markdown

@tyilo tyilo commented May 20, 2025

I think this is the correct behavior.

Also see #5364

@jieyouxu jieyouxu added S-waiting-on-review Status: awaiting review from the assignee but also interested parties. and removed pr-not-reviewed labels Feb 21, 2026
@rustbot

This comment has been minimized.

@tyilo tyilo force-pushed the empty-file-0-lines branch from 0efeead to 842f3fd Compare March 13, 2026 16:49
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Mar 13, 2026

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
Copy link
Copy Markdown
Collaborator

rustbot commented Mar 27, 2026

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

@ytmimi
Copy link
Copy Markdown
Contributor

ytmimi commented Mar 27, 2026

Closing since we merged #6839

@ytmimi ytmimi closed this Mar 27, 2026
@rustbot rustbot removed the S-waiting-on-review Status: awaiting review from the assignee but also interested parties. label Mar 27, 2026
@tyilo
Copy link
Copy Markdown
Author

tyilo commented Mar 27, 2026

@ytmimi Sad that the new PR got priority over this one, but at least it's fixed 🤷

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants