Glavin001/atom-beautify

Coffeescript support

Glavin001 opened this issue · 7 comments

just open a coffeescript file and press ctrl alt b... nothing happens

It "works" in some files. For instance:

Before

hello   derek how is  it going?
for c, i in "Hello  World!"
  k = 1+1-  2>=3<=  4>5   <6

  for c, i  in "Hello  World"
    k = (a,b)-> if b?  return a

f  =  b()[0]
for c, i in "Hello World"
  f(b())

After

hello derek how is it going?
for c, i in "Hello  World!"
  k = 1 + 1 - 2 >= 3 <= 4 > 5 < 6

  for c, i in "Hello  World"
    k = (a, b) -> if b? return a

f = b()[0]
for c, i in "Hello World"
  f(b())

The problem is not Atom Beautify's support of CoffeeScript Beautification -- it is always beautifying when requested. The problem is that Coffee-Formatter seems to be not to work in all cases and misses quite a lot of indentation and other problems unless it is very obvious (see above example).

The source code is in Literate CoffeeScript and looks fairly well documented. If you have some known issues that you'd like to beautify / format however it is currently not working, potentially submitting a Pull Request to the Coffee-Formatter would be great.

it uses default indentation for editor instead of default identation for coffeescript package?

Right now beautify makes 4 spaces indent , making linters unhappy.

it uses default indentation for editor instead of default identation for coffeescript package?

I'm not quite sure what you mean by default indentation for coffeescript package. I am not sure if CoffeeScript itself has a recommended default indentation size, however for Atom Beautify all settings are extracted from user configuration and can easily be changed to be what you want, such as the default indentation for coffeescript package as you described.

You can see the order of prioritized configurations at https://github.com/Glavin001/atom-beautify#configuration
In addition to .jsbeautifyrc you are able to use .editorconfig and Atom package settings for Atom Beautify, which provides the easiest view and descriptions for each option.

Right now beautify makes 4 spaces indent , making linters unhappy.

Then you can simply change it: see https://github.com/Glavin001/atom-beautify#configuration

screen shot 2016-06-06 at 2 31 25 pm
and yet, it's still beautifying with 2 spaces of indent. What gives?

@ironyinabox please create a new issue and follow the Issue template structure.

More specifically, be sure to add a link to the required debug.md Gist which includes debugging information that answers our most commonly asked questions: https://github.com/Glavin001/atom-beautify/blob/master/ISSUE_TEMPLATE.md#how-to-create-debugmd-gist
Thank you.