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-pluginディレクトリがESLintの観測対象になっていない #2250

Open
3 tasks
MT224244 opened this issue Aug 25, 2024 · 1 comment
Open
3 tasks

Comments

@MT224244
Copy link
Contributor

内容

タイトルの通りです。
既に1箇所カンマ抜けがありますが、検出されていません。

(Bug ReportかFeature Requestか迷いましたが、アプリケーションの動作に問題があるわけではないのでこちらにしました。)

Pros 良くなる点

Cons 悪くなる点

実現方法

npm scriptsの lint fmt それぞれのディレクトリ指定に eslint-plugin を追加する。
もしくは、特に問題なければいっそディレクトリ指定を . にしてしまっても良い気がします。

VOICEVOXのバージョン

0.?.0

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

  • Windows
  • macOS
  • Linux

その他

@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 26, 2024

本当ですね!! issue作成ありがとうございます!

全体をeslintの対象にしちゃってもいい気はしつつ、dist以下も見に行かないかとか.node_moduleはどうかとか、あと.storybookは見に行くのかとか、意外と考慮事項があるかもです。

package.jsonに書き足すのは・・・ありだと思います!
ただちょっと氾濫しつつあるから一旦そうするという形になるかもです。
その時はディレクトリ構造を整理していくというissueを別途立てることになるかも。

あるいはbuild/ディレクトリの下に置くのもいいかもしれません。
ここに置かれてるファイルはビルドに使うというより、どっちかというと開発用って感じなので、適してそうだなと。

このうちどれかの方法で問題が解決したらissueはcloseで良さそう!

eslint-pluginを作ってくださったのは @cm-ayf さんなので、もしご興味あれば 🙏 )

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

No branches or pull requests

2 participants