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

feat: tailwindcssのlintを導入 #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: tailwindcssのlintを導入 #30

wants to merge 2 commits into from

Conversation

mathsuky
Copy link
Collaborator

@mathsuky mathsuky commented Sep 17, 2024

忙しそうだったのでtailwindcssのlint入れるPR出しちゃいました。すでに作業してたらごめんなさい。
基本はプラグインのデフォルトの設定ですが,以下の2点を変更してあります。

  • クラス名の整頓を必須にしました。
  • 任意値の使用に対してwarningを出すようにしました。

任意値の使用をwarnにした理由としては,そもそもの目的である「表現に制限を加える」ということを達成したい一方,常に使わないで済ますことができない可能性があると考えたからです。ことによってはerrorでも良いかなーと考えているので,そのあたり意見をいただければと思います。(Pugmaくん as リーダー,てぃだくん as デザイン, その他有識者)

@Pugma
Copy link
Collaborator

Pugma commented Sep 19, 2024

取り急ぎコメント残しますが、せっかく今から ESLint 使うなら、 v9 以降の flat config を使ったほうがいいんじゃないかなって感じました
時間あればやってみてほしいかも (依存関係的にきつかったら教えてほしい)

@YuHima03 YuHima03 removed their request for review September 20, 2024 00:38
@mathsuky
Copy link
Collaborator Author

おそらく,vue/eslint-config-typescripteslintのv9に対応してなさそうです

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

Successfully merging this pull request may close these issues.

2 participants