tailwindcss-intellisense icon indicating copy to clipboard operation
tailwindcss-intellisense copied to clipboard

fix: bump `enhanced-resolve` for upstream fix

Open llllvvuu opened this issue 1 year ago • 2 comments

EDIT: Sorry, I just noticed https://github.com/webpack/enhanced-resolve/pull/301 still hasn't been merged. In that case there would be a need to rebase enhanced-resolve-301 instead of this.

This pulls in https://github.com/webpack/enhanced-resolve/pull/353.

Otherwise, the following is broken (https://github.com/webpack/enhanced-resolve/issues/352):

module.exports = require('@acme/ui/tailwind.config.cjs')

where @acme/ui has:

"exports": {
  "./*.cjs": "./*.cjs",
},

and requires the following workaround

"exports": {
  "./tailwind.config.cjs": "./tailwind.config.cjs",
  "./*.cjs": "./*.cjs",
},

Bumping the package fixes this. The tailwindcss-language-server code has a comment referencing https://github.com/webpack/enhanced-resolve/issues/282, but that's been fixed upstream so I'm not sure if it's still a blocker for this.

llllvvuu avatar Feb 14 '24 10:02 llllvvuu

Sorry, I just noticed https://github.com/webpack/enhanced-resolve/pull/301 still hasn't been merged. In that case there would be a need to rebase enhanced-resolve-301 instead of this @bradlc .

llllvvuu avatar Feb 14 '24 10:02 llllvvuu

Hey, thanks for this — I'll take a look soon!

thecrypticace avatar Feb 14 '24 21:02 thecrypticace

Closing in favor of #971 — looks like enhanced-resolve has merged the fix for the PnP issuer bug.

thecrypticace avatar May 29 '24 13:05 thecrypticace