Fat arrow is necessary when super is called

[plaxdan]

    run: ->
      When(someFunction()).done =>
        super

The above code triggers the 'unnecessary fat arrow' linter. However without the fat arrow, this within the super run will refer to window instead of the superclass.

The suggestion is that the linter checks for the presence of super in the fat arrow function and if it's present, then the linter passes. If no reference is found to either this or super then the fat arrow is indeed unnecessary.

[AsaAyers]

That's exactly how it works, except the check for super is missing https://github.com/clutchski/coffeelint/blob/master/src/rules/no_unnecessary_fat_arrows.coffee#L3

[plaxdan]

How do you run the tests? I'd like to fix it and add a pull request.

[AsaAyers]

npm install
npm test
On Jan 16, 2014 7:54 PM, "Daniel Walker" notifications@github.com wrote:

How do you run the tests? I'd like to fix it and add a pull request.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/214#issuecomment-32569523
.

[darky]

It's not fixed still :(