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
Fix sourceMappingURL for bundles with multiple entry points [CSS] #2867
Conversation
Have updated desciptions and added tests. Currently failing due to #2842. |
Now consistent with the logic in |
@mischnic Noticed the check for the sourcemap comment was using JS syntax rather than CSS. |
We really need to get CI working again. Very picky comment: could you rename the vars in that test? (from |
Has this been merged?!? |
I'm running into this exact issue and it appears this was approved, but closed instead of merged (unless I'm mistaken). Was that an accident? |
↪️ Pull Request
Fixes incorrect sourcemap URL's when you have multiple entry points.
Related issue #2851
💻 Examples
Given the followong CLI command
parcel build src/a/style.css src/b/style.css
The following files are produced
The sourcemap URLs are incorrect they should be
/a/style.css.map
and/b/style.css.map
.With the fix in place we instead get:
The same issue was previously fixed with JavaScript sourcemaps #2645
✔️ PR Todo