Skip to content

Pass through sourcemaps#23

Closed
mohsen1 wants to merge 2 commits intoVa1:masterfrom
mohsen1:patch-1
Closed

Pass through sourcemaps#23
mohsen1 wants to merge 2 commits intoVa1:masterfrom
mohsen1:patch-1

Conversation

@mohsen1
Copy link

@mohsen1 mohsen1 commented Mar 23, 2017

This will pass down the source maps to other loaders. It will produce a slightly less accurate SourceMap because source is changed but map hasn't. It's possible to fix that but it requires more work which I don't have time to fix it.

For anyone who's interested in using this fork:

You can install it via:

npm i -D 'mohsen1/string-replace-loader#sourcemap'

Fixes #18 #23

@mohsen1 mohsen1 closed this Mar 23, 2017
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.

Sourcemaps seem to break when replacing "ngInject" prologues.

1 participant