Skip to content

chore: bump enhanced-resolve version #971

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 29, 2024

Conversation

43081j
Copy link
Contributor

@43081j 43081j commented May 21, 2024

Builds on #907 but bumps the version of enhanced-resolve to latest.

Also drops a replace which should now be solved inside enhanced-resolve itself.

tests seem to pass but i'm not aware of a specific one to test what the replace was fixing/working around. if someone can point me in the right direction, im happy to add one

llllvvuu and others added 2 commits May 21, 2024 22:43
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.
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

Ah fantastic they've finally merged the fix. I'm gonna to do some testing to verify that things work and will get this merged probably tomorrow.

@thecrypticace
Copy link
Contributor

I tried to create a project similar to what was being fixed originally but it doesn't work in the existing extension 🙃

This doesn't seem to break other things though so I'm going to roll with it for now and will keep an eye out for PnP related issues. Thanks!

@thecrypticace thecrypticace merged commit f029cf2 into tailwindlabs:master May 29, 2024
@43081j 43081j deleted the enhanced-resolve branch May 29, 2024 16:25
@43081j
Copy link
Contributor Author

43081j commented May 29, 2024

thanks for taking a look 🙏

if any issues do come up, give me a shout and ill be happy to help

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.

3 participants