Skip to content
This repository has been archived by the owner on Jun 14, 2018. It is now read-only.

prevent newlines being added by OS command stdout #49

Closed
wants to merge 1 commit into from
Closed

prevent newlines being added by OS command stdout #49

wants to merge 1 commit into from

Conversation

xh3b4sd
Copy link

@xh3b4sd xh3b4sd commented Oct 21, 2016

This PR fixes #48.

@xh3b4sd
Copy link
Author

xh3b4sd commented Oct 21, 2016

I will fix the tests and add a new one in case we agree on the desire to have this change merged in.

@joefitzgerald
Copy link
Owner

Atom already adds a new line and trims spurious newlines from the end of a file on save. Are you really seeing more than 1 blank line at the end of a file?

@xh3b4sd
Copy link
Author

xh3b4sd commented Oct 25, 2016

The whitespace package has an option to ensure a trailing empty new line. I
disabled this. All files are fine, except the ones formatted using the
gofmt package in Atom. Is it maybe because of the package execution order?
Maybe the whitespace plugin kicks in before gofmt. In any case, the current
behavior is annoying. Do you experience another behaviour? Do your go files
not have trailing empty new lines when using this package here?

On Fri, 21 Oct 2016, 18:30 Joe Fitzgerald, [email protected] wrote:

Atom already adds a new line and trims spurious newlines from the end of a
file on save. Are you really seeing more than 1 blank line at the end of a
file?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#49 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhvQWoFaYhPeN74vPZjiwnf9s-nOvajks5q2OiRgaJpZM4KdUzS
.

@xh3b4sd
Copy link
Author

xh3b4sd commented Dec 2, 2016

I think this PR is wrong and does not fix anything. I am closing this now. Thanks for listening.

@xh3b4sd xh3b4sd closed this Dec 2, 2016
@xh3b4sd xh3b4sd deleted the no-newline branch December 2, 2016 10:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

gofmt produces empty new line
2 participants