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

v7 #173

Merged
merged 97 commits into from
Feb 3, 2022
Merged

v7 #173

merged 97 commits into from
Feb 3, 2022

Conversation

manovotny
Copy link
Owner

@manovotny manovotny commented Oct 18, 2021

Below is the list of changes in the upcoming release. It will be updated as new commits are made.

Highlights

  • ESLint 8 support!
  • Zero-config Next.js linting!
  • Zero-config TypeScript linting!

Breaking

  • New rules were added, which could potentially break builds.
  • Requires node 12.20.0 or higher.
  • Requires eslint 8.7.0 or higher. (Updates to ESlint 8. #175)
  • React filename extensions should be .jsx now instead of .js to more explicitly call out which files contain React and which files are pure JavaScript.
    • renamer can be used to aid in this conversion. For example: npx renamer --find js --replace jsx "components/**" --dry-run (remove --dry-run once everything looks good).
    • Probably update your NPM scripts to use eslint . --ext .js,.jsx.
  • package.json property ordering for consistency across projects.

Added

Changed

Fixed

  • Fixes false positives with import/extensions and mixed CommonJS / ESM usage.
  • Fixes false positives with React JSX transform rules for older versions of React.

Removed

@bskiff
Copy link

bskiff commented Jan 28, 2022

@manovotny, I think the engines listed in package.json don't correctly reflect the versions of node that this works on:
#181

@manovotny manovotny merged commit c4c3780 into main Feb 3, 2022
@manovotny manovotny deleted the v7 branch February 3, 2022 20:47
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.

2 participants