Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repeated arguments fail #199

Closed
dcousens opened this issue Jul 21, 2015 · 5 comments

Comments

@dcousens
Copy link
Member

commented Jul 21, 2015

Almost definitely an eslint bug, just filing it here.

function fun () {}

// expected Error about the missing space, instead it just fails and dies
fun(function(err, _, _, what) {

})
@dcousens

This comment has been minimized.

Copy link
Member Author

commented Jul 21, 2015

@feross could you confirm? Then I'll file a bug in eslint.

@feross

This comment has been minimized.

Copy link
Member

commented Jul 21, 2015

@dcousens confirmed

@dcousens dcousens self-assigned this Jul 21, 2015

@feross

This comment has been minimized.

Copy link
Member

commented Jul 22, 2015

Filed an issue: eslint/eslint#3128 :)

@dcousens

This comment has been minimized.

Copy link
Member Author

commented Jul 23, 2015

Thanks @feross, sorry.

@feross

This comment has been minimized.

Copy link
Member

commented Jul 24, 2015

@dcousens No worries.

It turns out this is expected behavior whenever eslint encounters a "parse error". We can attempt to convince the eslint team to treat errors like this as non-fatal so the rest of the linter errors will show up. But this may take a while for them to fix, so I don't want to track it here. Interested folk can follow the eslint bug here: eslint/eslint#3128

@feross feross closed this Jul 24, 2015

@lock lock bot locked as resolved and limited conversation to collaborators May 11, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
2 participants
You can’t perform that action at this time.