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

ESLint は v9 へのバージョンアップの準備ができていないので、当面の間は対象外とする。 #29

Merged
merged 3 commits into from
Apr 22, 2024

Conversation

s-sasaki-0529
Copy link
Contributor

ESLint は v9 から flat config がデフォルト化しますが、typescript-eslint を始めとする各種プラグインの対応が出揃わないとバージョンアップするメリットがないため、それまでの間は renovate の対象外とします。

"^eslint-",
"^prettier-"
]
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ESLint v9 の準備ができるまでは、グルーピングしていた eslint 系パッケージと Prettier は単独でバージョンアップするようにします。

],
"matchPackagePatterns": [
"^reg-"
]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ついでですが、既に reg-suit は社内で使われておらず chromatic に統一されてるので削除しちゃいます。

default.json Outdated
"^reg-"
]
"matchPackageNames": ["eslint"],
"enabled": false
Copy link
Contributor Author

Choose a reason for hiding this comment

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

eslint は問答無用で対象外に。
JSON なので意図のコメントを書けないのが辛い。

Copy link

@mkmn mkmn Apr 22, 2024

Choose a reason for hiding this comment

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

description fieldが使えると思います。
https://docs.renovatebot.com/configuration-options/#description

Copy link
Contributor Author

Choose a reason for hiding this comment

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

便利!
コメント入れときます!

@s-sasaki-0529 s-sasaki-0529 self-assigned this Apr 22, 2024
@s-sasaki-0529 s-sasaki-0529 marked this pull request as ready for review April 22, 2024 03:50
@s-sasaki-0529 s-sasaki-0529 requested review from a team, ytakahashi05 and goiso5 and removed request for a team April 22, 2024 03:51
Copy link

@ytakahashi05 ytakahashi05 left a comment

Choose a reason for hiding this comment

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

enabled フィールドの使い方を確認して、問題なさそうなのでapproveします🍎

https://docs.renovatebot.com/configuration-options/#enabled

リマインドも設定されているので、設定を戻し忘れることもなさそう 👍

https://kufuinc.slack.com/archives/CJEUZ4MS6/p1713424676743249

Copy link

@goiso5 goiso5 left a comment

Choose a reason for hiding this comment

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

よさそうでした!

@s-sasaki-0529 s-sasaki-0529 merged commit bd3769c into master Apr 22, 2024
1 check passed
@s-sasaki-0529 s-sasaki-0529 deleted the ignore-eslint branch April 22, 2024 07:56
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.

4 participants