Non-UNIX line endings causes verification to fail about grunt-jsbeautifier HOT 9 CLOSED

Unfortunately we just facing the same problem here.

from grunt-jsbeautifier.

I have pushed a fix in version 0.2.9, see #45 & #46 Let me know if this does fix for you.

from grunt-jsbeautifier.

Unfortunately this didn't fix it.
If you have an formatted js file with UNIX line endings (LF) and run the jsbeautifier task everything is fine. But if you change the line endings to Windows (CR LF) und run the task again, it will change the line endings back to LF.
This causes uncommited changes in GIT, but GIT will ignore the changes if you try to stage them. IntelliJ on Windows also has problems with that and shows all "beautified" files as changed.
Do you know what causes the line endings change?
Thx for your help.

from grunt-jsbeautifier.

Having the same issue on Windows, our git is setup to checkout with Windows line endings and commit back with Unix line endings. All the files come up as changed on git even when no changes are made. This is preventing us from using this as a part of our workflow, Maybe we should log this in https://github.com/beautify-web/js-beautify ?

from grunt-jsbeautifier.

There is a bug open in beautifier/js-beautify#260

I am also doing new line check in this plugin but its only for back compatibility. This check in plugin was added when new line support from js-beautify was limited.

from grunt-jsbeautifier.

+1 - facing the same issue with git... we can't use the plugin as is. It doesn't look like there is any traction on the referenced open js-beautify issue from almost a year and a half ago.

from grunt-jsbeautifier.

beautifier/js-beautify#260 ... Fixed.

from grunt-jsbeautifier.

@everyone,
First of all apologies for not following up on this. But looks like js-beautify bug is fixed. Will you please check and confirm if that fixed the issue for you as well. If it does then report here so I can close the ticket.

Thanks a lot in advance.

from grunt-jsbeautifier.

Not heard for long back, hope issue is fixed so closing ticket. If this issue still persist, please open new bug.

from grunt-jsbeautifier.