-
Notifications
You must be signed in to change notification settings - Fork 134
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
Please add support for zackad/prettier-plugin-twig-melody
#254
Please add support for zackad/prettier-plugin-twig-melody
#254
Comments
Oh sweet — I'll take a look in a few! |
Ah they haven't published a separate package and are relying users putting in the GitHub url the dependency version. I think it'd probably best to wait until they publish this as a scoped package with version published to npm. I'll keep an eye on it though to see if that happens. |
Gonna close for now but will absolutely take a look once they do |
hey @thecrypticace thanks for picking this up, I've asked the author of the fork if they can do that, I'll keep you posted! |
Other than publishing as npm package, what do i need to make my fork to be compatible with |
and @thecrypticace FYI the npm package is here (thanks @zackad !) |
Nah, your fork works almost as is. We don't currently support plugins that simultaneously support Prettier 2 and 3 but it's an easy thing to support. (the signature for |
I've re-added support and will get a release out today! |
bless you both, thank you so much! |
@thecrypticace sorry to necro this but I think it's good that you have some context. Let me know if you'd rather I opened a new issue. @zackad just updated the package and renamed it from https://www.npmjs.com/package/@zackad/prettier-plugin-twig Thank you! |
Ah okay cool — will do. I opened a separate issue for me so I don't forget. I probably won't get to it until this evening though. |
No rush at all, take your time man! Thank you. |
Hi @alessandro-newzoo, any updates on this? |
@alessandro-newzoo @bitbirddev Please take a look at this: https://github.com/ttskch/prettier-plugin-tailwindcss-anywhere |
Hi Takashi and thanks for this, very interesting! |
@alessandro-newzoo are you using an oder version of @zackad/prettier-plugin-twig(-melody)? |
@bitbirddev yeah sorry for the confusion, I just realized. I'm currently using @zackad/[email protected] (a), which is the last version before it was renamed to @zackad/prettier-plugin-twig (b). (a) is compatible with To clarify:The plugin originally known as https://www.npmjs.com/package/@zackad/prettier-plugin-twig-melody (a) has been renamed to https://www.npmjs.com/package/@zackad/prettier-plugin-twig (b). Currently:
|
@alessandro-newzoo got you. it think the only options for now is to just wait until #327 is merged. i've wasted too much time already to find a workaround :P |
@bitbirddev haha yeah, it's been a very long, confusing and frustrating journey, but thanks to zackad and your PR, hopefully this will be the end of it (assuming they'll ever merge it 😅) |
you can already use it if you want to. in your project directory run
|
Thank you! I'd rather wait as I have a very complicated setup and I'd rather stick to live packages if possible, but I'll keep this in mind should tailwindlabs decide to never merge your PR 🤞🏻 |
Hello,
The plugin trivago/prettier-plugin-twig-melody was abandoned a long time ago and therefore it does not support Prettier 3.x (This plugin used to be supported by older versions of
prettier-plugin-tailwindcss
).BUT, this awesome person - god bless them - forked it, and made it work with Prettier v3: zackad/prettier-plugin-twig-melody
If it's not too much work, could you guys please add support for it? I know it's small and has no stars yet, but it works perfectly with v3, and I can guarantee you it's the only Prettier plugin on the internet working with v3 at the moment.
I'm currently stuck with Prettier 2.x and
prettier-plugin-tailwindcss
v0.4.1
, so this would really help me (and other people working with Twig + TailwindCSS) tremendously.Thank you,
Alessandro
The text was updated successfully, but these errors were encountered: