Skip to content

Conversation

dominicm00
Copy link
Contributor

This plugin uses tailwindcss internally, but marks it only as a dev dependency rather than a peer dependency, causing errors/undefined behavior in monorepos and Yarn PNP. Correctly adding tailwind as a peer dependency allows for correct resolution to occur while still referencing the user's tailwind and not an internal version.

Other official plugins have already corrected this: see tailwindlabs/tailwindcss-forms@8a8671b

This pull request is identical to tailwindlabs/tailwindcss-line-clamp#6. Both other official plugins (forms, typography) already correctly use peer dependencies.

dominicm00 and others added 2 commits May 10, 2021 20:02
This plugin uses tailwindcss internally, but marks it only as a dev dependency rather than a peer dependency, causing errors/undefined behavior in monorepos and Yarn PNP. Correctly adding tailwind as a peer dependency allows for correct resolution to occur while still referencing the *user's* tailwind and not an internal version.

Other official plugins have already corrected this: see tailwindlabs/tailwindcss-forms@8a8671b

This pull request is identical to tailwindlabs/tailwindcss-line-clamp#6. Both other official plugins (forms, typography) already correctly use peer dependencies.
@adamwathan
Copy link
Member

Thank you!

@adamwathan adamwathan merged commit 6e1871a into tailwindlabs:master May 24, 2021
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