search

lkytal / coffeelinter

Coffee Lint extension for VS Code
GNU Affero General Public License v3.0
7 stars 9 forks source link

Erroneous "unexpected newline" after super call with no parentheses #5

Closed zcoffin86 closed 6 years ago

zcoffin86 commented 6 years ago

I'm not sure if this functionality is deprecated or otherwise discouraged in coffeescript in general, however it seems to still be valid so...

As discussed in various places (such as https://stackoverflow.com/questions/5953670/extend-class-and-pass-all-constructor-arguments-to-super), this is accepted syntax, therefore should not be flagged as a lint error?

http://recordit.co/MizXjMKQ3s 2018-05-24 13_16_20- test coffee - untitled workspace - visual studio code 2018-05-24 13_16_36- test coffee - untitled workspace - visual studio code

zcoffin86 commented 6 years ago

actually i guess this is a parse error. Bare super was apparently deprecated in CoffeeScript 2: https://coffeescript.org/#breaking-changes-super-extends. Strange though, I'm using the latest coffee script (2.3.0) and the arguments are actually passed... I'm already updating with the new syntax. Sorry for the confusion. Cheers!

lkytal commented 6 years ago

Thanks for your feedback I just notice it. You may find more info from the linter itself http://www.coffeelint.org/.