Skip to content
This repository was archived by the owner on Oct 7, 2024. It is now read-only.

Update ESLint config to v11#181

Merged
Gudahtt merged 2 commits intomainfrom
update-eslint-config-v11
Dec 16, 2022
Merged

Update ESLint config to v11#181
Gudahtt merged 2 commits intomainfrom
update-eslint-config-v11

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Dec 15, 2022

The ESLint config has been updated to v11, and all lint-related packages have been updated.

Most changes were fixed using yarn lint:fix. A manual change was needed to remove use of the in operator, and to ignore the lint warning for the use of the events API. A note was added to the README to explain that the events API is used.

The lint:misc script was updated to include a new flag, to better match the module template.

@Gudahtt Gudahtt force-pushed the update-eslint-config-v11 branch 2 times, most recently from 5af3f57 to b8be87d Compare December 15, 2022 14:20
@Gudahtt Gudahtt marked this pull request as ready for review December 15, 2022 14:20
@Gudahtt Gudahtt requested a review from a team as a code owner December 15, 2022 14:20
@Gudahtt Gudahtt force-pushed the update-eslint-config-v11 branch 2 times, most recently from 5282555 to 25d0885 Compare December 15, 2022 14:24
Gudahtt and others added 2 commits December 15, 2022 20:05
The ESLint config has been updated to v11, and all lint-related
packages have been updated.

Most changes were fixed using `yarn lint:fix`. A manual change was
needed to ignore the error for use of the `in` operator, and to ignore
the lint warning for the use of the `events` API. A note was added to
the README to explain that the `events` API is used.

The `lint:misc` script was updated to include a new flag, to better
match the module template.
Co-authored-by: Elliot Winkler <elliot.winkler@gmail.com>
@Gudahtt Gudahtt force-pushed the update-eslint-config-v11 branch from 42722fb to 99db463 Compare December 15, 2022 23:35
Copy link
Copy Markdown
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt Gudahtt merged commit 0a69ce3 into main Dec 16, 2022
@Gudahtt Gudahtt deleted the update-eslint-config-v11 branch December 16, 2022 14:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants