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

Better support native class property definitions (and update to ESLint v8 internally) #1529

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

bmish
Copy link
Member

@bmish bmish commented Jul 20, 2022

Includes:

  • Handles both node types ClassProperty (ESLint v7) and MethodDefinition (ESLint v8) in various rules/helpers
  • A few mandatory linting dependency updates and autofixed violations
  • Ensures both ESLint v7 and ESLint v8 are tested

Part of v11 release (#1169).

@bmish bmish force-pushed the eslint-8 branch 2 times, most recently from c7f25b4 to 7f9ace4 Compare July 20, 2022 21:42
@bmish bmish changed the title Update to ESLint v8 Update to ESLint v8 and better support native class property definitions Jul 20, 2022
@bmish bmish changed the title Update to ESLint v8 and better support native class property definitions Better support native class property definitions (and update to ESLint v8 internally) Jul 20, 2022
@bmish bmish mentioned this pull request Jul 20, 2022
@bmish bmish marked this pull request as ready for review July 20, 2022 21:55
@bmish bmish merged commit d10c1be into master Jul 20, 2022
@bmish bmish deleted the eslint-8 branch July 20, 2022 21:56
@bmish bmish changed the title Better support native class property definitions (and update to ESLint v8 internally) Better support native class property definitions Jul 20, 2022
@bmish bmish changed the title Better support native class property definitions Better support native class property definitions (and update to ESLint v8 internally) Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant