明示的にimportされていないcomponentsがある・componentsのimport手法にブレがあることに関する対処 - Githubissues

Tokyo-Metro-Gov / covid19

東京都新型コロナウイルス感染症対策サイト / Tokyo COVID-19 Task Force website

https://stopcovid19.metro.tokyo.lg.jp/

MIT License

6.26k stars 1.97k forks source link

明示的にimportされていないcomponentsがある・componentsのimport手法にブレがあることに関する対処 #6021

Closed y-chan closed 3 years ago

y-chan commented 3 years ago

改善詳細 / Details of Improvement

5982 で、MonitoringCommentCard.vue内で元々明示的にimportされていて、現在も使用されているcomponent(AppLink)のimportが外されてしまいました。現状、このプロジェクトではcomponentsのオートインポートが適用されているおかげで何の支障もなく動作していますが、一部IDEでは警告表示が出ますし、あくまで私の主観かもしれませんが、可読性に欠けるとも思います。
LazyComponentsに関しては仕方がないと思いますが、せめて普通に動作しているcomponentsだけでも統一して明示的にインポートすべきではないかなと考えます。
また、先ほど挙げたPRで、import手法が相対パスであり、こちらもプロジェクト最上層からの絶対パスに統一すべきと考えます
今回の提案はリファクタであり、機能追加等の提案ではないので、優先順位は低めです。

kaizumaki commented 3 years ago

@y-chan ご提案ありがとうございます！AppLinkが使用されているにも関わらずimportが削除されているのは、私のチェックミスでした。こちらのリファクタのPRを出していただくことはできますか？

y-chan commented 3 years ago

OKです、後で出しておきます！

munierujp commented 3 years ago

相対パスによるインポートについては、ESLintで警告を出すようにできそうですね。

kaizumaki commented 3 years ago

@munierujp フォローありがとうございます！たしかに、eslint通すのはよさそうですね。