-
Notifications
You must be signed in to change notification settings - Fork 135
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
Typescript maps for debug/coverage #39
Comments
@marcmeans - any ides on this? |
let me take a peek and see. |
I so badly need this. Have you given karma-sourcemap-loader a try @lathonez and didn't work or are you just keeping notes? |
I started to look at it. But with the upgrade to beta4 in ionic2 I've gotten distracted. I have this on my list to look at in the next week still. If I can get it working I'll let you know. |
@marcmeans awesome, please keep us posted. Will take a look myself after I upgrade my project to the latest libraries to see if I can get any insights. |
@masimakopoulos - I forget where that reference come from, someone on this repo was using it but I don't know to what degree of success. @marcmeans - appreciate that you're still trying to look at this. I've been busy recently. As of last night everything in this repo should be latest. If you have no time let me know and I'll pick it up |
I've just used it briefly tonight in the 'lightweightify' branch on this @lathonez https://github.com/lathonez I see the repo — |
@marcmeans - I'm kinda working on this as a by-product of #68 |
@lathonez aren't the maps being generated by ionic-gulp-browserify-typescript? I see this in their source code:
and a few lines later
So if I am understanding this correctly, source maps are generated if browserify debug mode is true. Which from the same file
seems to be true by default. |
Yeah this is fine when bundling the spec. Not sure if we're going down that
|
As per #68 sourcemaps are now available @ |
No idea how to implement this!
The text was updated successfully, but these errors were encountered: