Renovate のパッケージアップデートの場合はコード全体に ESLint を実行する

Tokyo-Metro-Gov / covid19

東京都新型コロナウイルス感染症対策サイト / Tokyo COVID-19 Task Force website

https://stopcovid19.metro.tokyo.lg.jp/

MIT License

6.26k stars 1.97k forks source link

Renovate のパッケージアップデートの場合はコード全体に ESLint を実行する #5221

Open shgtkshruch opened 4 years ago

shgtkshruch commented 4 years ago

改善詳細 / Details of Improvement

GitHub Actions で reviewdog を実行して、PR 単位で ESLint のチェックをしています
Renovate で ESLint 関連のパッケージアップデートする場合、現状の reviewdog の仕組みでは ESLint が漏れる可能性があります
- reviewdog のデフォルト設定では PR の diff に対するリントのみをフィルタリングするため

スクリーンショット / Screenshot

期待する見せ方・挙動 / Expected behavior

Renovate の PR に関しては、コード全体に ESLint を実行すること
1. 開発環境で実行している yarn lint
2. reviewdog の no-filter オプションでも可能かもしれません（未テスト）

動作環境・ブラウザ / Environment

GitHub Actions

goki90210 commented 4 years ago

ESLintだけではなくprettierも対象にしてもらえますか？(#5141 参照) prettierのメジャーアップグレードで大量にエラーが発生したので…。

shgtkshruch commented 4 years ago

このプロジェクトでは、ESLint の設定に prettier のルールセットが入っているので、通常通り ESLint を実行すればそちらも対象になると思います。

goki90210 commented 4 years ago

@shgtkshruch

このプロジェクトでは、ESLint の設定に prettier のルールセットが入っているので、通常通り ESLint を実行すればそちらも対象になると思います。

そういう話ではなく、

ESLint はアップグレードせずに
prettierのみアップグレードされた場合

にコード全体に ESLint を実行されないのではないかというお話です。

kaizumaki commented 4 years ago

RenovateがeslintのマイナーアップデートのPR #5278 を立てていますが、これはマージしても大丈夫でしょうか...？

goki90210 commented 4 years ago

@kaizumaki ローカル環境で試しましたが問題なさそうです。

kaizumaki commented 4 years ago

@goki90210 ありがとうございます！助かります 🙏 ツール系のライブラリは、何をどのように判断してアップデートすべきか迷いますね...

kaizumaki commented 4 years ago

prettierのマイナーアップデートのPR #5333 がきましたが、これはどのようにしたらいいでしょうか...？私のローカル環境で試したところ、yarn lint するといくつかエラーを吐くようです。こちらのissueが解決されるまで、#5333 は放置orクローズしたほうがいいでしょうか。あるいはマージ後に yarn lint を通したものをPRするか。ルールのほうを修正するということも考えられますかね。参考：prettier 2.1.0のリリースノート