Skip to content
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

chore(webpack-config-compass): Replace exports.webpack with custom exports key COMPASS-5724 #2990

Merged

Conversation

gribnoysup
Copy link
Contributor

exports.webpack helped us bundle Compass from source, but at the same time
it was messing with external users of published packages, specifically cloud
team was having troubles with it and had to add special aliases in their webpack
configuration to work around the issue

Using custom main and exports fields completely avoids the issue as these will
be ignored by any other webpack configuration unless explicitly specified

…ports key

`exports.webpack` helped us bundle Compass from source, but at the same time
it was messing with external users of published packages, specifically cloud
team was having troubles with it and had to add special aliases in their webpack
configuration to work around the issue

Using custom main and exports keys completely avoids the issue as these will
be ignored by any other webpack configuration unless explicitly specified
@gribnoysup
Copy link
Contributor Author

gribnoysup commented Apr 14, 2022

Huh, seems like there is a bunch of check issues that fell through the cracks somehow and now that I touched literally every package, they all surfaced Okay, nope, just a weird config in compass-home

@gribnoysup gribnoysup merged commit 2cd4eaf into main Apr 14, 2022
@gribnoysup gribnoysup deleted the compass-5724-replace-exports-webpack-with-custom-exports-key branch April 14, 2022 12:34
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.

None yet

2 participants