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

deps: Bump eslint suite to v9 #1565

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

deps: Bump eslint suite to v9 #1565

wants to merge 2 commits into from

Conversation

0b5vr
Copy link
Contributor

@0b5vr 0b5vr commented Dec 27, 2024

This PR bumps eslint suite to v9.

This also migrates the existing eslint configs to the new flat config.

See the commit logs for details.

Migrated the eslint config using `npx @eslint/migrate-config .eslintrc.json`, and did not work flawlessly, obviously

I had to include config to examples into the root config file.
Instead, we are able to delete `.eslintrc` that existed for each example directory though

`require-atomic-updates` (has been set to `off`) does not exist in the default configs, so removed the line

`@typescript-eslint/no-unnecessary-type-assertion` requires a type assertion, removed it for now
@0b5vr 0b5vr added dependencies Pull requests that update a dependency file Refactor Cool code is cool labels Dec 27, 2024
@0b5vr 0b5vr added this to the next milestone Dec 27, 2024
@0b5vr 0b5vr self-assigned this Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Refactor Cool code is cool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant