Skip to content

Conversation

@llllvvuu
Copy link
Contributor

@llllvvuu llllvvuu commented Feb 14, 2024

EDIT: Sorry, I just noticed webpack/enhanced-resolve#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 webpack/enhanced-resolve#353.

Otherwise, the following is broken (webpack/enhanced-resolve#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 webpack/enhanced-resolve#282, but that's been fixed upstream so I'm not sure if it's still a blocker for this.

Otherwise, the following is broken:
```javascript
module.exports = require('@acme/ui/tailwind.config.cjs')
```

where `@acme/ui` has:
```json
"exports": {
  "./*.cjs": "./*.cjs",
},
```

and requires the following workaround
```json
"exports": {
  "./tailwind.config.cjs": "./tailwind.config.cjs",
  "./*.cjs": "./*.cjs",
},
```

bumping the package fixes this.
@llllvvuu llllvvuu force-pushed the fix/bump_enhanced_resolve branch from a1e2e40 to 87792ca Compare February 14, 2024 10:05
@llllvvuu llllvvuu marked this pull request as draft February 14, 2024 10:07
@llllvvuu
Copy link
Contributor Author

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

@thecrypticace thecrypticace self-assigned this Feb 14, 2024
@thecrypticace
Copy link
Contributor

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

43081j added a commit to 43081j/tailwindcss-intellisense that referenced this pull request May 21, 2024
Builds on tailwindlabs#907 but bumps the version of enhanced-resolve to latest. Also
drops a `replace` which should now be solved inside enhanced-resolve
itself.
43081j added a commit to 43081j/tailwindcss-intellisense that referenced this pull request May 21, 2024
Builds on tailwindlabs#907 but bumps the version of enhanced-resolve to latest. Also
drops a `replace` which should now be solved inside enhanced-resolve
itself.
@thecrypticace
Copy link
Contributor

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

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