Skip to content

Add a tailwindcss:minify task #138

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

Closed
wants to merge 1 commit into from
Closed

Add a tailwindcss:minify task #138

wants to merge 1 commit into from

Conversation

cbldev
Copy link

@cbldev cbldev commented Feb 3, 2022

Thanks to official Tailwind CSS documentation, with "Tailwind CLI, you can minify your CSS by adding the --minify flag" (source: https://tailwindcss.com/docs/optimizing-for-production).

Can we have first a minify task like this proposition on the tailwindcss-rails gem?

#greenit

Thanks to official Tailwind CSS documentation, with "Tailwind CLI, you can minify your CSS by adding the --minify flag" (source: https://tailwindcss.com/docs/optimizing-for-production).

Can we have first a minify task like this proposition on the tailwindcss-rails gem?

#greenit
@cbldev
Copy link
Author

cbldev commented Feb 3, 2022

#136

@svenfinger svenfinger mentioned this pull request Feb 9, 2022
@dixpac
Copy link
Contributor

dixpac commented Feb 12, 2022

Thanks @cyrildotcc !
How would you setup production build to minify the output with minify task that you have added in this PR?

@dhh
Copy link
Member

dhh commented Feb 12, 2022

We should just minify all the time. Browsers unfold that anyway for debugging.

@dixpac
Copy link
Contributor

dixpac commented Feb 16, 2022

We should just minify all the time. Browsers unfold that anyway for debugging.

@cyrildotcc are you interested in working on this 🙏

@dhh
Copy link
Member

dhh commented Feb 19, 2022

Fixed via c76f5d2

@dhh dhh closed this Feb 19, 2022
@cbldev
Copy link
Author

cbldev commented Feb 23, 2022

@dixpac yes, another time maybe: @dhh did the job 🎉

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.

3 participants