Please add support for `zackad/prettier-plugin-twig-melody`

[alessandro-newzoo]

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

[thecrypticace]

Oh sweet — I'll take a look in a few!

[thecrypticace]

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.

[thecrypticace]

Gonna close for now but will absolutely take a look once they do

[alessandro-newzoo]

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!

[zackad]

Other than publishing as npm package, what do i need to make my fork to be compatible with prettier-plugin-tailwindcss? Do I need to convert my fork into ES module or it's not a requirement?

[alessandro-newzoo]

and @thecrypticace FYI the npm package is here (thanks @zackad !)
https://www.npmjs.com/package/@zackad/prettier-plugin-twig-melody

[thecrypticace]

Other than publishing as npm package, what do i need to make my fork to be compatible with prettier-plugin-tailwindcss? Do I need to convert my fork into ES module or it's not a requirement?

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 parse() changed in v3 so that options is the 2nd argument not 3rd).

[thecrypticace]

I've re-added support and will get a release out today!

[alessandro-newzoo]

bless you both, thank you so much!

[alessandro-newzoo]

@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 @zackad/prettier-plugin-twig-melody to @zackad/prettier-plugin-twig - could you pretty please update the plugin to support the new package name? 🙏🏻

https://www.npmjs.com/package/@zackad/prettier-plugin-twig

Thank you!