`insert_final_newline` property about atom-editorconfig

[florianb]

Okay, this functionality is implemented by the atom-whitespace-package. Since this is a core-package its functionality has to be treated as first-class-citizen of Atom.

The good message about this is, we're able to easily add support for insert_final_newline, by modifying the text-buffer on the onWillSave-event.

The bad message is, the whitespace-package may override a "negative" setting, since it may be called after our hook.

I'll try to find a solution with the package-maintainer.

[sindresorhus]

@JaKXz Of course it isn't. It's not implemented yet. Hence this issue...

[anthonyryan1]

Subscribing. Interested in this for insert_final_newline = false on hand-optimized SVG files (saving as many bytes as possible).

[kevinSuttle]

Any reason you recommend setting this to false for Markdown files?

[sindresorhus]

I don't. I do for #3 though, since Markdown requires to trailing spaces for linebreak.

[JaKXz]

It appears that this setting isn't taking effect for me on Atom 0.169.0 with v0.2.3 running on OSX 10.10.1.

[JaKXz]

Oh. That's odd, I could swear I'd seen it working before [and I had disabled Atom's whitespace plugin]... but obviously I was just seeing things. Sorry for the noise.

[jerone]

@JaKXz commented on 10 jan. 2015 06:31 CET:

Oh. That's odd, I could swear I'd seen it working before [and I had disabled Atom's whitespace plugin]... but obviously I was just seeing things. Sorry for the noise.

atom-beautify supports this:

Js end with newline

[sindresorhus]

@EloB Please read the thread before commenting... #4 (comment)

[mahnunchik]

Any news?

[florianb]

Our feature-request is not supported. It is suggested to work around the issue, which is what we will do. I assume to get this issue into WIP in the next days.

I am sorry for the inconvenience - i was some weeks away from desk and i am still getting back. :)