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

[shgtkshruch]

改善詳細 / 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]

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

[shgtkshruch]

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

[goki90210]

@shgtkshruch

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

そういう話ではなく、

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

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

[kaizumaki]

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

[goki90210]

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

[kaizumaki]

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

[kaizumaki]

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