lint-stagedでeslintのwarnを許容しなくしたら何故かprettierが終わらなくなったので直した

@typescript/use-unused-vars を error にはしたくないけど、コミットには含めたくなかったので、これをやった

そうしたら何故か prettier --write が永遠に終わらなくなったので直した話。

結論

files を prettier --write にも渡す

.lintstagedrc.js

// @see https://github.com/okonet/lint-staged/issues/1017

module.exports = {
  '**/*.{ts,tsx,js,jsx}': async files => {
    const filesToLint = await removeIgnoredFiles(files);
    return [`prettier --write ${filesToLint}`, `eslint --max-warning=0 ${filesToLint}`]
  }
}

詳細

もともとの設定がこんな感じだった

package.json

{
  "lint-staged": {
    "**/*.{ts,tsx,js,jsx}": ["prettier --write", "eslint"]
  }
}

これをwarnを許容しないようにするために設定を書き換えた。書く場所も package.json ではなく .lintstagedrc.js に変更。

.lintstagedrc.js

// @see https://github.com/okonet/lint-staged/issues/1017

module.exports = {
  '**/*.{ts,tsx,js,jsx}': async files => {
    const filesToLint = await removeIgnoredFiles(files);
    return ['prettier --write', `eslint --max-warning=0 ${filesToLint}`]
  }
}

これで動かしたら prettier --write が永遠に終わらなくなったので、色々調べてたら上にあげたissueにたどり着いた。

Since you are using the function syntax, it's assumed to pass the files yourself:

https://github.com/okonet/lint-staged/issues/1017#issuecomment-924312570

とのこと ほえ〜