Skip to content
New issue

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

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

Already on GitHub? # to your account

Set sourceMapContents option by default if sourceMaps are set. #104

Closed
wants to merge 1 commit into from
Closed

Set sourceMapContents option by default if sourceMaps are set. #104

wants to merge 1 commit into from

Conversation

at0g
Copy link

@at0g at0g commented May 23, 2015

Fixes #92

@jhnns
Copy link
Member

jhnns commented Jun 3, 2015

Thanks for the pull-request, but I'm not sure if I should merge it. It increases the size of the final css considerably and thus is not an option to use in production...

As long as your able to set sourceMapContents=true in your app, that's a fair workaround.

@jhnns
Copy link
Member

jhnns commented Aug 3, 2015

After fiddling around, I realized that webpack includes the sourceMapContents by default anyway. Thus I've finally merged this manually. Shipped with 1.0.4.

Thx for your effort! 👍

@jhnns jhnns closed this Aug 3, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sourcemaps are empty
2 participants