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!: Update ESLint version requirements #354

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

wakamsha
Copy link
Contributor

@wakamsha wakamsha commented Dec 2, 2024

What changed / motivation ?

To support ESLint v9, most plugins need to be updated. This will also change the minimum required version of ESLint.

Linked PR / Issues

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Chore (routine task, maintenance, or non-functional change that doesn't modify src or test files)

Pre-flight checklist

References

@wakamsha wakamsha self-assigned this Dec 2, 2024
@wakamsha wakamsha force-pushed the feat/eslint-version-requirements branch from 91a03a9 to 50f3283 Compare December 2, 2024 09:38
@wakamsha wakamsha marked this pull request as draft December 3, 2024 00:55
@wakamsha
Copy link
Contributor Author

wakamsha commented Dec 7, 2024

This PR is frozen as a Draft until #338 is merged.

@wakamsha wakamsha force-pushed the feat/eslint-version-requirements branch from 50f3283 to fc1fa32 Compare December 9, 2024 01:51
@wakamsha wakamsha marked this pull request as ready for review December 9, 2024 07:44
@wakamsha wakamsha marked this pull request as draft December 9, 2024 08:40
@wakamsha
Copy link
Contributor Author

wakamsha commented Dec 9, 2024

This PR is frozen as a Draft until #355 is merged.

@wakamsha wakamsha force-pushed the feat/eslint-version-requirements branch from a93d119 to 7c7e949 Compare December 9, 2024 09:56
@wakamsha wakamsha force-pushed the feat/eslint-version-requirements branch from 7c7e949 to 630a9c5 Compare December 9, 2024 10:00
@wakamsha wakamsha marked this pull request as ready for review December 9, 2024 10:03
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems that the output of rule settings has changed since ESLint was upgraded to v9.

In v8, all options were output, but in v9, unspecified options are no longer output. Since these two options were not specified and left at the default values, it seems that they are no longer output.

@@ -55,15 +55,15 @@
"eslint-plugin-react-hooks": "^5.0.0",
"eslint-plugin-storybook": "^0.11.1",
"eslint-plugin-testing-library": "^7.0.0",
"eslint-plugin-unicorn": "^56.0.0",
"eslint-plugin-unicorn": "^56.0.1",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

When used in combination with ESLint v9.15.0 or higher, lint will fail, so you need to update eslint-plugin-unicorn.

see: sindresorhus/eslint-plugin-unicorn#2497

Copy link
Member

@taigakiyokawa taigakiyokawa left a comment

Choose a reason for hiding this comment

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

LGTM 🙆‍♂️ Thank you!

@wakamsha wakamsha merged commit 78aa086 into main Dec 10, 2024
2 checks passed
@wakamsha wakamsha deleted the feat/eslint-version-requirements branch December 10, 2024 01:07
oss-frontend-github-app bot pushed a commit that referenced this pull request Dec 10, 2024
## [5.0.0](4.1.2...5.0.0) (2024-12-10)

### ⚠ BREAKING CHANGES

* Update ESLint version requirements (#354)
* Update TypeScript Rules and Config to support ESLint v9 (#338)
* Update eslint-plugin-next version (#353)
* Update eslint-plugin-testing-library version (#351)
* Update eslint-plugin-react-hooks version (#349)
* Update eslint-plugin-jsdoc version (#346)
* Update eslint-plugin-promise version (#345)
* Update eslint-plugin-import version (#344)
* Add a ruleset powered by `unicorn` plugin into `essentials` config (#335)

### Features

* Add a ruleset powered by `unicorn` plugin into `essentials` config ([#335](#335)) ([c5a1544](c5a1544))
* Update ESLint version requirements ([#354](#354)) ([78aa086](78aa086))
* Update eslint-plugin-import version ([#344](#344)) ([37927c4](37927c4))
* Update eslint-plugin-jest version ([#342](#342)) ([5beb961](5beb961))
* Update eslint-plugin-jest-dom version ([#343](#343)) ([e803148](e803148))
* Update eslint-plugin-jsdoc version ([#346](#346)) ([9e0cf74](9e0cf74))
* Update eslint-plugin-jsx-a11y version ([#348](#348)) ([85f8fe9](85f8fe9))
* Update eslint-plugin-n version ([#347](#347)) ([3d78df4](3d78df4))
* Update eslint-plugin-next version ([#353](#353)) ([e906b76](e906b76))
* Update eslint-plugin-promise version ([#345](#345)) ([e7abb32](e7abb32))
* Update eslint-plugin-react version ([#350](#350)) ([6c8551c](6c8551c))
* Update eslint-plugin-react-hooks version ([#349](#349)) ([22f5358](22f5358))
* Update eslint-plugin-storybook version ([#352](#352)) ([2171394](2171394))
* Update eslint-plugin-testing-library version ([#351](#351)) ([ca2bcec](ca2bcec))
* Update TypeScript Rules and Config to support ESLint v9 ([#338](#338)) ([b945312](b945312))
@oss-frontend-github-app
Copy link

🎉 This PR is included in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants