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

Add Tailwind CSS Standalone CLI utility filter #42

Merged
merged 8 commits into from
Jul 8, 2024

Conversation

bennothommo
Copy link
Member

This PR adds support for using the Tailwind CSS standalone CLI utility as a filter in Assetic, allowing the building of Tailwind CSS through Assetic.

To allow this filter to work correctly, some adjustments have been made to the base Process filter class:

  • The binary path of the filter can be specified on the fly now using the setBinaryPath method.
  • The base directory that the process will run in can also be changed using the setWorkingDirectory method. This allows the Tailwind CSS utility to run within a certain folder, allowing the config to be determined automatically and for content files to be correctly found and parsed.

@bennothommo bennothommo requested a review from a team as a code owner July 8, 2024 03:29
@LukeTowers
Copy link
Member

@bennothommo looks good! Feel free to merge and release when ready.

Are you doing this in order to add core support for Tailwind to the Winter CMS Asset Combiner or just using it for a different project?

@LukeTowers
Copy link
Member

@bennothommo one final thing needed is to add it to the list of filters in the README.

@bennothommo
Copy link
Member Author

@LukeTowers honestly, it's just me finally getting around to doing some stuff I've been talking about for ages 😂

@LukeTowers
Copy link
Member

@bennothommo carry on then, much appreciated ❤️

@bennothommo bennothommo merged commit fde0642 into master Jul 8, 2024
7 checks passed
@bennothommo bennothommo deleted the wip/tailwindcss-standalone-filter branch July 8, 2024 05:05
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