Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

型情報付きのESLintを有効化する #1895

Closed
3 tasks
sevenc-nanashi opened this issue Mar 3, 2024 · 3 comments · Fixed by #2172
Closed
3 tasks

型情報付きのESLintを有効化する #1895

sevenc-nanashi opened this issue Mar 3, 2024 · 3 comments · Fixed by #2172

Comments

@sevenc-nanashi
Copy link
Member

内容

型情報が必要なルールを有効化したいです。

Pros 良くなる点

  • バグが減る。

Cons 悪くなる点

  • Lintが遅くなる。
  • 現状警告が大量に出るのでそれを対処する必要がある。

実現方法

.tsファイルなら楽そうなんですけどVueだと厳しそうですね...:vuejs/eslint-config-typescript#29

VOICEVOXのバージョン

0.?.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

(なし)

@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 17, 2024

導入したいですね!!

昔やろうとしたことがあるのですが、修正が大変すぎて諦めました。
とりあえずwarningで導入してみるとかありかもですね!
ちなみにその時のPRこちらです。@typescript-eslint/recommended-requiring-type-checkingを導入しようとしていました。

.tsファイルなら楽そうなんですけどVueだと厳しそうですね

上のプルリクエストではワークアラウンドで迂回してるので参考になるかもです。
tsconfig.json.eslintrc.jsを変更しています。
僕が試した感じだと問題になってたのが.jsにtypescript用のESLintが当たってしまうことだったので、.ts.vueにだけ当たるようにしています。
ちゃんと知らずやってるので、もっとうまい設定があるのかもですが・・・。

@Hiroshiba
Copy link
Member

もしかしたら @cm-ayf さんが詳しいかも・・・?

@cm-ayf
Copy link
Contributor

cm-ayf commented Mar 17, 2024

あまり詳しくないです、私も.vueでESLintを使ったことはここ以外で無くて……

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

Successfully merging a pull request may close this issue.

3 participants