Skip to content
This repository has been archived by the owner on May 18, 2022. It is now read-only.

Lintチェックが遅い #37

Open
tamuraryoya opened this issue Sep 28, 2019 · 7 comments
Open

Lintチェックが遅い #37

tamuraryoya opened this issue Sep 28, 2019 · 7 comments

Comments

@tamuraryoya
Copy link
Contributor

確実に原因はeslint-plugin-importにあるが、Airbnbなどは特に重たくないため、異様に重たくするルールをOnにしている可能性が高い。

要調査&検証。

@tamuraryoya
Copy link
Contributor Author

あるときの実行結果

Promiseがawaitせずに存在していると警告する@typescript-eslint/no-floating-promisesが圧倒的に遅い。

image

@tamuraryoya
Copy link
Contributor Author

tamuraryoya commented Nov 29, 2019

@typescript-eslint/no-floating-promisesとったら今度は@typescript-eslint/await-thenableが台頭してきた

image

@tamuraryoya
Copy link
Contributor Author

import/no-cycleも大概時間くってるっぽい

image

@tamuraryoya
Copy link
Contributor Author

問題ありそうなルールを削除しても190秒を切ることがなかった。もっと他に原因があるのかも

image

@tamuraryoya
Copy link
Contributor Author

Algoriaのルールに差し替え(+Prettier無効化)たら、もう30秒くらい縮められそうだった

image

@tamuraryoya
Copy link
Contributor Author

遅い上位のルールを無効化したが時間自体は変わらなかった

image

@tamuraryoya
Copy link
Contributor Author

遅い理由の考察:

  • そもそもルールの設定数が多すぎる
    • 地理も積もれば理論
  • 対して効果もないのにちょっと遅いルールをいっぱい仕込んでいる
    • AirbnbとかAlgoriaとかで設定されてないルールの理由

ただ、no-floating-promises以外はもはや誤差レベル(時間も増えたり減ったりする)ので3分ちょっとの壁は超えられない気がする。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant