Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Update eslint-config-standard and switch from -jsx to -react #1465

Merged
merged 1 commit into from
Sep 24, 2019
Merged

Update eslint-config-standard and switch from -jsx to -react #1465

merged 1 commit into from
Sep 24, 2019

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Sep 9, 2019

Updates the ESLint standardjs presets to latest, and switches from eslint-config-standard-jsx to eslint-config-standard-react, since the latter is a superset of the former and required for full React support.

The new configs set the React version and plugins for us, so we no longer need to set them ourselves.

Fixes #1449 and fixes #1450.

Updates the ESLint standardjs presets to latest, and switches from
`eslint-config-standard-jsx` to `eslint-config-standard-react`, since
the latter is a superset of the former and required for full React
support.

The new configs set the React version and plugins for us, so we no
longer need to set them ourselves.

Fixes #1449 and fixes #1450.
@edmorley edmorley added this to the Neutrino 9 milestone Sep 9, 2019
@edmorley edmorley self-assigned this Sep 9, 2019
@edmorley edmorley requested a review from a team September 9, 2019 19:13
@edmorley
Copy link
Member Author

@neutrinojs/core-contributors? :-)

@timkelty
Copy link
Contributor

@edmorley I'll take a look this morning!

@edmorley
Copy link
Member Author

@timkelty? :-)

@timkelty
Copy link
Contributor

Whoops, thanks for the nudge. Looking now!

Copy link
Contributor

@timkelty timkelty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 LGTM!

@edmorley
Copy link
Member Author

Thank you :-)

@edmorley edmorley merged commit d8b0ee1 into neutrinojs:master Sep 24, 2019
@edmorley edmorley deleted the update-eslint-config-standard branch September 24, 2019 05:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

2 participants