-
Notifications
You must be signed in to change notification settings - Fork 13
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
Solution for #4 #5
base: master
Are you sure you want to change the base?
Conversation
…rs (mostly useful for debugging
This is a very useful feature and it's awesome that @askarby put in the time to submit this PR. Is there any way we can work on moving this forward? |
@askarby This was my fault. I was still calling it with a single map file instead of pointing at a directory. This works great when you call it correctly 😛 |
Great job! Yes this PR contains a lot of unrelated changes, so someone would need to pull out the essential stuff needed to support multiple maps. I'll look into it when I have time |
Hi folks! Looking forward for this landed! |
Feel free to make a new PR with only the feature in question, and I can look into it. |
@mifi Any chance to get that merged in and distribute npm package ? This would be helpful a lot! |
Does anyone have some sample source map files and minified stacktraces that reference more than one source map? Now that this code is about to become more complex I would like to write an integration test for multi-source map |
This is a PR to support multiple source-map files (support for resolving stacktraces based on webpack chunks, eg. Angular etc.)
Solves:
#4
Please Notice: I did feature-creep a bit, while developing, so you might want to "pick'n'choose" from what you want to integrate (I added "debug"-support, "legend"-support and webstorm IDE-settings).