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.

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

[Bug] make tailwind should be called tailwind-install instead #326

Open
LarsArtmann opened this issue Nov 12, 2024 · 3 comments
Open

[Bug] make tailwind should be called tailwind-install instead #326

LarsArtmann opened this issue Nov 12, 2024 · 3 comments
Labels
good first issue Good for newcomers

Comments

@LarsArtmann
Copy link
Contributor

What is the problem?

make tailwind feels like it is there to create an output.css file.
But it only installs tailwind if you do not already have it.
As with make templ-install, we should be consistent with the naming.

Operating System

ALL

Architecture Version (x86, x64, arm, etc)

ALL

Steps to reproduce

run make tailwind

Relevant log output

None
@Ujstor
Copy link
Collaborator

Ujstor commented Nov 16, 2024

We can change the target naming to be consistent with the operation it performs.

@Ujstor Ujstor added the good first issue Good for newcomers label Nov 20, 2024
@pradytpk
Copy link

pradytpk commented Nov 21, 2024 •
edited
Loading

I would to Like to work on this issue.
Can you please assign me.
@Ujstor @LarsArtmann

pradytpk added a commit to pradytpk/go-blueprint that referenced this issue Nov 21, 2024
@pradytpk
make tailwind should be called tailwind-install instead Melkeydev#326
139029a
pradytpk added a commit to pradytpk/go-blueprint that referenced this issue Nov 21, 2024
@pradytpk
make tailwind should be called tailwind-install instead Melkeydev#326
c1e7080
@pradytpk pradytpk mentioned this issue Nov 21, 2024
Open
2 tasks
pradytpk added a commit to pradytpk/go-blueprint that referenced this issue Nov 21, 2024
@pradytpk
make tailwind should be called tailwind-install instead Melkeydev#326 …
2e4ba2f 
…- updated the documentation
@Ujstor Ujstor assigned Ujstor and unassigned Ujstor Nov 21, 2024
@Ujstor
Copy link
Collaborator

Ujstor commented Nov 21, 2024

This is yours. I will check the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@LarsArtmann @pradytpk @Ujstor