fix jsx-no-bind to address any function binding (not just methods)

palantir / tslint-react

:orange_book: Lint rules related to React & JSX for TSLint.

Apache License 2.0

749 stars 76 forks source link

fix jsx-no-bind to address any function binding (not just methods) #252

Closed amir-arad closed 4 years ago

amir-arad commented 4 years ago

I've encountered a code that contains function binding of the form: < ... onConfirm={execute.bind(null, someProp)} > IMO jsx-no-bind should alert on this

palantirtech commented 4 years ago

Thanks for your interest in palantir/tslint-react, @amir-arad! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

adidahiya commented 4 years ago

Sorry, tslint-react is deprecated and I'm not accepting any new features / enhancements: https://github.com/palantir/tslint-react/issues/210. Look into eslint-plugin-react instead.