Skip to content

Conversation

@dominicm00
Copy link
Contributor

@dominicm00 dominicm00 commented Jun 2, 2021

What's the problem this PR addresses?
tailwindcss currently does not work with yarn PnP due to issues with a dependency and a couple plugins. This PR adds the necessary compat data for tailwindcss and all the official plugins to work out of the box.

How did you fix it?
For the official tailwindcss extensions, I submitted PRs to add the correct peer dependencies (tailwindlabs/tailwindcss-aspect-ratio#14, tailwindlabs/tailwindcss-line-clamp#6) that have since been accepted, and added compat data for older versions. There is also a specific version of postcss-purgecss that was missing a peer dependency for postcss, and tailwindcss is pinned to that version for compatibility reasons, so I added compat data for the specific version in question.

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

Dominic Martinez added 3 commits June 2, 2021 15:36
Two of the official tailwindcss plugins implicitly used tailwindcss as a peer dependency but didn't list it; they have since been fixed, and this adds compatibility for earlier versions.

See tailwindlabs/tailwindcss-aspect-ratio#14, tailwindlabs/tailwindcss-line-clamp#6
@fullhuman/postcss-purgecss removed postcss as a peer dependency for version `3.1.3`, which was incorrect and it was added back in the next release, `4.0.0`. Unfortunately, tailwindcss is stuck on this version of postcss-purgecss for compatibility reasons, so this commit adds compatibility for @fullhuman/postcss-purgecss@3.1.3.

See FullHuman/purgecss@ff2eff5, FullHuman/purgecss@24116f3,
merceyz and others added 2 commits June 3, 2021 10:50
`@fullhuman/postcss-purgecss@3.1.3` had a prerelease tag `alpha.0` that needed to be added.

Co-authored-by: Kristoffer K. <merceyz@users.noreply.github.com>
@dominicm00 dominicm00 requested a review from merceyz June 8, 2021 21:37
Copy link
Member

@merceyz merceyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I can't merge it without the CI passing, which is why I didn't merge it earlier, so will let @arcanis do the honours.

@merceyz merceyz enabled auto-merge (squash) June 10, 2021 18:21
@merceyz merceyz merged commit 5ddfce3 into yarnpkg:master Jun 10, 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