search

clutchski / coffeelint

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

newline_after_classes is still acting a bit weird #518

Open swang opened 9 years ago

swang commented 9 years ago

See https://github.com/clutchski/coffeelint/commit/03b82446a3cc720b78b5df94d40e6daae6003ee5#commitcomment-13728928 for initial comment about this.

startswithaj commented 9 years ago

I put the whole file in a gist >>

https://gist.github.com/startswithaj/ef410b67bcd80b9a272f

screen shot 2015-10-13 at 5 24 17 pm

Not in the screen shot I have 28 lines.

startswithaj commented 9 years ago

screen shot 2015-10-13 at 5 26 55 pm

VictorHagsand commented 9 years ago

Also still have strange issues with this.

jekuno commented 8 years ago

I can reproduce the error with a stripped down example:

class MyClass
  myFunction: ->
    someVar = "someText"

    someVar

Produces the error #4: Wrong count of newlines between a class and other code. Expected 3 got 2.

The stripped down coffelint.json:

{
  "newlines_after_classes": {
    "value": 3,
    "level": "warn"
  }
}

The error disappears after removing the blank line in the function body.