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

InstallTailwind #18

Merged
merged 2 commits into from
Sep 4, 2024
Merged

InstallTailwind #18

merged 2 commits into from
Sep 4, 2024

Conversation

Pugma
Copy link
Collaborator

@Pugma Pugma commented Sep 3, 2024

@Pugma Pugma requested a review from mathsuky September 3, 2024 16:19
Copy link
Collaborator

@mathsuky mathsuky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。気になったことを書きます。

  • 公式docの5番とかのoutput.css回りが適切に設定されていなさそう。
  • 直接は関係ないですが,リンター/フォーマッターの設定を共有したいですね。それをやるならば,このままぷぐまくんに作業していただくか,適切にタスクを振ってもらえると良いと思います。言い出しっぺなのに申し訳ないのですが,僕はサークル外のタスクとAPI設計が落ち着くまでは厳しいです:sry:

@Pugma
Copy link
Collaborator Author

Pugma commented Sep 4, 2024

Tailwind に関しては、リンクとして提示した Vite 用の導入手順を参照しました
css に追記するとそれは main.ts 経由で index.html に読み込まれるので、index.html や output.css の出力は必要ないと考えてます

僕も初めて使うので詳しくはないけど、 npm run dev した限りは正常に css が適用されてました

@mathsuky mathsuky dismissed their stale review September 4, 2024 13:02

俺の勘違い

Copy link
Collaborator

@mathsuky mathsuky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

良さそうです,ありがとうございます
リンターなどの設定を引き続きここでやるならApprove取り下げときます。

@mathsuky
Copy link
Collaborator

mathsuky commented Sep 4, 2024

僕の見間違いでした:sry:
設定の方は上にも書いたけど,ここで引き続きやるか別のブランチでやるかなど,担当割含めてよろしくお願いいたします。

@Pugma
Copy link
Collaborator Author

Pugma commented Sep 4, 2024

Linter は僕そこまで詳しくないけど、まぁできる範囲でやってみます
一応適切な PR 粒度のためにこれはマージして別ブランチでやっちゃいます
担当割はちょっとだけデザインを確認してから割り振ります

@Pugma Pugma merged commit d9eec1a into main Sep 4, 2024
4 checks passed
@Pugma Pugma deleted the installTailwind branch September 4, 2024 13:50
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