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.

Sign up for GitHub

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

Jump to bottom

ESLintを導入する #192

Open

jdkfx opened this issue Mar 6, 2024 · 2 comments

Labels

Contributor

jdkfx commented Mar 6, 2024 •

edited

Loading

内容

現状、Linterが導入されていないことと様々な人が参加するため、人によりコードの書き方などがバラバラになっている
ESLintを導入し、コーディング規約に則っているかチェックしてもらいたい

Pros 良くなる点

コーディング規約に沿ったコードを書くことで、コードのミスやレビューの手間を減らす
今後の実装コストや運用コストにかける負荷を減らすことができる
新しくコントリビュータが参加しやすくなる

Cons 悪くなる点

導入後にこれまで書かれてきたコードを変更する必要がある
規約を決めるにあたり、その内容を議論する必要がある

実現方法

公式ドキュメントに書かれている内容など、参考にできる情報があるので実現できそう
https://www.gatsbyjs.com/docs/how-to/custom-configuration/eslint/

VOICEVOXのバージョン

0.16.1

その他

関連Issueとして、 importのformatterを導入する #22 が挙げられる

jdkfx added the 機能向上 label

Member

Hiroshiba commented Mar 6, 2024 •

edited

Loading

とても良いと思います！！！！！！
ぜひ導入したいです･･･！！！

Contributor

wappon28dev commented Mar 6, 2024 •

edited

Loading

ESLint 良いですね！

ちなみにですが, こちら (#163 (comment)) で一度あったのですがうっかり key を設定し忘れてエラーになっちゃったなどあったので, React や TypeScript のアンチパターンを未然に防げるようなルールセットの選定も考慮できると良いかもですね！

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