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

Check newline at the end of the file #27

Closed
wants to merge 9 commits into from

Conversation

AucaCoyan
Copy link
Contributor

Hi! It's me again. I added the check of the new line at the end of the file.
If, for some reason, the output of nufmt doesn't end on a new line, add it.

I remember somebody asking for a config that opt out of this feature. I research some popular formatters and all of them add a new line at the end of the file. Is there a case where is not recommended?
\n

@AucaCoyan AucaCoyan mentioned this pull request Jun 9, 2023
22 tasks
@amtoine
Copy link
Member

amtoine commented Jun 14, 2023

there are a bunch of duplicate commmits here with #26, namely 4b97279ad6822b916dba66a3ea1272387d0cc371..c1f18dd1d6e3e4512a12dbd8f442108f19a378fa but we're really only interested in 581f3b37e5398c5b2046d64820c8be9871b76e19..1b6120865c3571722c119bf213295d9488e14edf 😮

i doubt we can properly open a PR on top of another unmerged one in GitHub...

@AucaCoyan can you reopen this either

?

@amtoine amtoine closed this Jun 14, 2023
fdncred pushed a commit that referenced this pull request Jun 15, 2023
I resolved my git and did properly the #27 PR 😄
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.

None yet

2 participants