next.js
cef8bfa5 - Revert "refactor: remove unnecessary `lint-staged` operations (#65861)" (#66554)

Commit
1 year ago
Revert "refactor: remove unnecessary `lint-staged` operations (#65861)" (#66554) cc @samcx ### Why? The previous change on `lint-staged` dropped, ignoring files from eslint, resulting in errors on files that should be ignored (e.g. `dist` inside `.github/actions/`). x-ref: #66383 ### How? This PR reverts PARTIAL commit 0558f61c415e8456edd6c22ec3be137c8959d564. ### RFC > ESLint v8.51.0 introduced [--no-warn-ignored CLI flag](https://eslint.org/docs/latest/use/command-line-interface#--no-warn-ignored). It suppresses the warning File ignored because of a matching ignore pattern. Use "--no-ignore" to override warning, so manually ignoring files via eslint.isPathIgnored is no longer necessary. How about we convert current config as flat config, which would be necessary if we upgrade to v9, and use the `--no-warn-ignored` flag on `lint-staged` to reduce the operation introduced in #65861? x-ref: https://github.com/lint-staged/lint-staged?tab=readme-ov-file#eslint--8510--flat-eslint-config --------- Co-authored-by: Sam Ko <sam@vercel.com>
Author
Parents
Loading