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

Support ESLint 3 and new rule format #661

Closed
paazmaya opened this issue Jul 5, 2016 · 5 comments
Closed

Support ESLint 3 and new rule format #661

paazmaya opened this issue Jul 5, 2016 · 5 comments

Comments

@paazmaya
Copy link

paazmaya commented Jul 5, 2016

The new rule format, introduced in the ESLint version 3, is explained here http://eslint.org/blog/2016/07/eslint-new-rule-format

Any chance of converting the rules in that format and making a new major release which support ESLint 3 and up?

@ljharb
Copy link
Member

ljharb commented Jul 5, 2016

The only point of doing so would be to ensure compatibility with a future eslint v4, and doing so would break compatibility with eslint v2.

What's the hurry?

@paazmaya
Copy link
Author

paazmaya commented Jul 6, 2016

Living on the edge....

@MoOx
Copy link
Contributor

MoOx commented Jul 8, 2016

The cool part is that there is a tool for that http://eslint.org/blog/2016/07/eslint-new-rule-format#automatic-transformation

@ljharb
Copy link
Member

ljharb commented Jul 8, 2016

I've recently learned that both eslint v2 and v3 support this new rule format - so I think that the sooner we convert all the rules, the better.

@paazmaya
Copy link
Author

Thanks for pointing that out @MoOx, I have seem not to read that far...

@yannickcr yannickcr mentioned this issue Jul 17, 2016
9 tasks
petersendidit added a commit to petersendidit/eslint-plugin-react that referenced this issue Jul 22, 2016
petersendidit added a commit to petersendidit/eslint-plugin-react that referenced this issue Jul 24, 2016
petersendidit added a commit to petersendidit/eslint-plugin-react that referenced this issue Jul 27, 2016
petersendidit added a commit to petersendidit/eslint-plugin-react that referenced this issue Jul 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants