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

Support activation for tailwind.config.cjs #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support activation for tailwind.config.cjs #139

wants to merge 1 commit into from

Conversation

lakuapik
Copy link

As issued here #133

Currently headwind doesnt automatically activate when using ESM file tailwind.config.cjs

This PR add activation events for file tailwind.config.cjs.

@lakuapik
Copy link
Author

@heybourn or @praveenperera do you mind checking on this? i see some people are upvoting this PR.

@vintprox
Copy link

vintprox commented Nov 4, 2021

Oh no, we're stuck here it seems.

@praveenperera
Copy link
Contributor

Hey I can check but unfortunately @heybourn hasn't been around to do new releases.

@hacknug
Copy link

hacknug commented Nov 15, 2021

Hey I can check but unfortunately @heybourn hasn't been around to do new releases.

I tried to contact him directly a while ago with no luck either. Should we look into publishing a new extension? Maybe Microsoft has a way for contributors to claim co-ownership of abandoned extensions so we can ship the update to our existing users?

@praveenperera
Copy link
Contributor

@hacknug do you want to start a fork and release a new version? I will also try and help

@vintprox
Copy link

vintprox commented Nov 15, 2021 via email

@vintprox
Copy link

vintprox commented Nov 15, 2021

I tried to contact Ryan via Open Collective page and sent a trick. I hope it will be resolved without more forks (imagine the confusion "Headwind" and "Footwind" would bring 😂).

@hacknug
Copy link

hacknug commented Nov 15, 2021

Let's give it a few days before forking in case Ryan gets back to us.

Maybe it would also be a good idea to discuss the idea of adding this to the official extension? It would make it easier for devs since they would only need to install one extension. This change could also help us tackle #5 which imho is one of the first things we should try to solve for v3 of this extension.

Thoughts on this @adamwathan @bradlc? I know this isn't an option for this year since you plan two months of work ahead, but maybe there's a way we can coordinate after TailwindCSS v3 is out and aim for a Q1 of 2022?

@adamwathan
Copy link

Yeah we have talked about doing this for years and Brad has prototyped it in the past, we've even done work in core to prepare for supporting it but the biggest blocker has always been finding a way to do it that wasn't hacky or without a lot of downsides. We have a proof-of-concept implementation as a Prettier plugin already but got hung up on something and can't remember exactly what it was 🤔 Definitely something hoping to sink our teeth into properly soon though.

@sweetroll
Copy link

@hacknug can't see Footwind in the marketplace yet ;)
@heybourn are you active now by any chance?

@n17r053v3n
Copy link

Hey, I have no idea if anyone is still active here cuz it looks like not.
I've been able to work my way out of using cjs and using mjs for those files that are js by default in root directory (sveltekit) but I have not been able to find a way to use it with cjs eve though it can be merged above i see no one with write access had merged it. Do you know of any other way to use tailwind formatting feature than this plugin? I have tried some prettier node extensions but i cannot make it work.
image

@hacknug
Copy link

hacknug commented Mar 7, 2023

Do you know of any other way to use tailwind formatting feature than this plugin? I have tried some prettier node extensions but i cannot make it work.

You can either use the official Prettier plugin for an config-less approach or look into eslint-plugin-tailwindcss if you want to configure the order of your classes.

@stormynight9
Copy link

Any updates?

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.

None yet

8 participants