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

Prepare to switch to eslint-plugin-react 6 #975

Closed
wants to merge 1 commit into from

Conversation

akozhemiakin
Copy link

@akozhemiakin akozhemiakin commented Jul 30, 2016

This pull request is applicable to eslint-plugin-react@^6.0.0-rc.0. So I think it will be a good idea
to keep it until switching to that version and update it along with new breaking changes in react plugin.

"pragma" parameter from "jsx-uses-react" and "react" parameter from "no-deprecated" must be removed because they were deprecated in eslint-plugin-react since version 5.* and removed in 6.0.0-rc.0. Also it is already specified in the shared section (settings.react.pragma) as it should be, so nothing to fix there.

"allow-in-func" parameter must be removed from "no-did-mount-set-state" and "no-did-update-set-state" because since 6.0.0-rc.0 it is the default behavior and "allow-in-func" parameter was removed.

"wrap-multilnes" was deprecated and "jsx-wrap-multilines" must be used instead.

@akozhemiakin akozhemiakin changed the title Fix react/jsx-uses-react rule Fix several react rules Jul 31, 2016
from "no-deprecated" must be removed because they were
deprecated in eslint-plugin-react since version 5.* and
removed in 6.0.0-rc.0. Also it is already specified in the
shared section (settings.react.pragma) as it should be, so
nothing to fix there.

"allow-in-func" parameter must be removed from
"no-did-mount-set-state" and "no-did-update-set-state" because
since 6.0.0-rc.0 it is the default behavior and
"allow-in-func" parameter was removed.

"wrap-multilnes" was deprecated and "jsx-wrap-multilines"
must be used instead.
@akozhemiakin akozhemiakin changed the title Fix several react rules Prepare to switch to eslint-plugin-react 6 Jul 31, 2016
@ljharb
Copy link
Collaborator

ljharb commented Jul 31, 2016

Thanks for the contribution, but I already have all the changes required to switch to eslint-plugin-react v6 locally. I do not want to switch to an rc version prior to v6 being released.

@ljharb ljharb closed this Jul 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants