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

feat!: upgrade eslint v8 (fixes #61) #99

Merged
merged 3 commits into from
Dec 16, 2021
Merged

feat!: upgrade eslint v8 (fixes #61) #99

merged 3 commits into from
Dec 16, 2021

Conversation

aladdin-add
Copy link

No description provided.

package.json Outdated Show resolved Hide resolved
@aladdin-add aladdin-add force-pushed the issue61 branch 3 times, most recently from b82bb60 to dd511f8 Compare November 23, 2021 10:02
the main change is to replace eslint.CliEngine to eslint.ESLint, as CLIEngine
has been removed.

refs: https://eslint.org/docs/8.0.0/user-guide/migrating-to-8.0.0#remove-cliengine

fix: rm no needed deps

these eslint plugins has been installed as eslint-config-react-app's deps.
@aladdin-add aladdin-add force-pushed the issue61 branch 4 times, most recently from 890fe07 to ceea0c4 Compare December 16, 2021 09:17
forked eslint-config-react-app v7.0.0, as it was breaking `dts lint`
@aladdin-add aladdin-add marked this pull request as ready for review December 16, 2021 09:42
@aladdin-add aladdin-add merged commit 63d37b9 into main Dec 16, 2021
@aladdin-add aladdin-add deleted the issue61 branch December 16, 2021 09:42
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.

1 participant