Empty lines before semi

[mightyiam]

Just a failing test case for what I would suggest to be a possible improvement.

[github-actions]

Nixpkgs diff

[piegamesde]

Why are you randomly adding failing test cases? If you have an issue with the formatter, please open one instead. Either way, the current behavior is by design

nixfmt/standard.md

Line 68 in 3bcb63c

- Line breaks may be added or removed, but empty lines must not be created. Single empty lines must be preserved, and consecutive empty lines must be collapsed into a single empty line.

#201

but personally, I'd be open to a discussion re-evaluating it

[mightyiam]

Did you chide me for opening a PR with a failing test case instead of an issue?

[piegamesde]

Yes. That test case doesn't properly follow the layout for tests btw. Also, these are snapshot tests and not unit tests. They are automatically generated and exist to track changes in the formatter, not to expect a specific output. Manually writing a test's output is bound to fail for most non-trivial inputs, and generally non-trivial tests which test as many corner cases as possible are preferred.

[jfly]

Closing this in favor of #298.

@mightyiam, please file GitHub issues for future formatting suggestions. This policy will be made clear in this upcoming README update.