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

Disable jsx-handler-name eslint rule #157

Merged
merged 1 commit into from
Jul 24, 2016
Merged

Disable jsx-handler-name eslint rule #157

merged 1 commit into from
Jul 24, 2016

Conversation

mxstbr
Copy link
Contributor

@mxstbr mxstbr commented Jul 24, 2016

Closes #156

Not sure if we can simply delete the line and that'll disable it, so I
went with setting it to 0. Let me know!

@ghost ghost added the CLA Signed label Jul 24, 2016
@gaearon
Copy link
Contributor

gaearon commented Jul 24, 2016

Just deleting will be fine.

Ref #156

Not sure if we can simply delete the line and that'll disable it, so I
went with setting it to 0. Let me know!
@mxstbr mxstbr force-pushed the jsx-handler-name branch from 58dfa77 to 3a707ea Compare July 24, 2016 11:12
@mxstbr
Copy link
Contributor Author

mxstbr commented Jul 24, 2016

Updated, thanks!

@gaearon gaearon merged commit 662fdb7 into master Jul 24, 2016
@mxstbr mxstbr deleted the jsx-handler-name branch July 24, 2016 15:13
@mxstbr mxstbr mentioned this pull request Jul 25, 2016
@gaearon gaearon added this to the 0.2.0 milestone Jul 25, 2016
@gaearon gaearon mentioned this pull request Jul 27, 2016
ianschmitz pushed a commit to ianschmitz/create-react-app that referenced this pull request Nov 5, 2017
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants