search

clutchski / coffeelint

Lint your CoffeeScript.
http://www.coffeelint.org
Other
1.18k stars 171 forks source link

no_spaces #556

Open JulianWielga opened 8 years ago

JulianWielga commented 8 years ago

Adds no_spaces rule. Related to #135

swang commented 8 years ago

I'll be on the road for the day so I haven't tried this out yet but do the tests actually cover everything? Specifically I'm worried about chained calls such that...

startingChain()
    .hello()
    .world()
    .today((x) -> x + 2)

Specifically .hello is indented with 4 spaces, .world and .today are indented with a tab. My guess(?) is that this may fail because coffeescript slurps the newline\spaces before .hello so it's on the same line as startingChain().

I'll be able to test this more later on, either tonight or after the weekend.

UziTech commented 4 years ago

I added this to @coffeelint/cli in https://github.com/coffeelint/coffeelint/pull/7. Thanks!