Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Sourcemaps support #109

Closed
mwisnicki opened this issue Oct 23, 2014 · 6 comments · Fixed by #146 · 4 remaining pull requests
Closed

Sourcemaps support #109

mwisnicki opened this issue Oct 23, 2014 · 6 comments · Fixed by #146 · 4 remaining pull requests
Labels
PR exists

Comments

@mwisnicki
Copy link

Can you add support for sourcemaps ?
@mwisnicki
Copy link
Author

This will need gotwarlost/istanbul#212 first.

@dotnetwise
Copy link

👍
Without sourcemaps when using browserify makes coverage useless.

@kuraga kuraga mentioned this issue Dec 10, 2014
Closed
@dayyan
Copy link

dayyan commented Jan 20, 2015

👍 It would be nice to see where in my coffee script source errors are occurring.

@GoulwenBZH
Copy link

👍 same thing for typescript code after tsify/browserify.

nmalaguti added a commit to nmalaguti/karma-coverage that referenced this issue Apr 20, 2015
@nmalaguti
feat(preprocessor): Add sourcemap support
4956e19 
Adds sourcemap support to the preprocessor. If the file being
processed has a sourcemap property, it is merged with the output
from the coverage instrumentation. This enables better debugging
on instrumented code.

Closes karma-runner#109
@deepsweet deepsweet mentioned this issue Apr 20, 2015
Closed
@nmalaguti nmalaguti mentioned this issue Apr 20, 2015
Merged
nmalaguti added a commit to nmalaguti/karma-coverage that referenced this issue Apr 20, 2015
@nmalaguti
feat(preprocessor): Add sourcemap support
a80300d 
Adds sourcemap support to the preprocessor. If the file being
processed has a sourcemap property, it is merged with the output
from the coverage instrumentation. This enables better debugging
on instrumented code.

Closes karma-runner#109
nmalaguti added a commit to nmalaguti/karma-coverage that referenced this issue Jun 9, 2015
@nmalaguti
feat(preprocessor): Add sourcemap support
d6234b8 
Adds sourcemap support to the preprocessor. If the file being
processed has a sourcemap property, it is merged with the output
from the coverage instrumentation. This enables better debugging
on instrumented code.

Closes karma-runner#109
nmalaguti added a commit to nmalaguti/karma-coverage that referenced this issue Jun 9, 2015
@nmalaguti
feat(preprocessor): Add sourcemap support
de3b738 
Adds sourcemap support to the preprocessor. If the file being
processed has a sourcemap property, it is merged with the output
from the coverage instrumentation. This enables better debugging
on instrumented code.

Closes karma-runner#109
@Izhaki Izhaki mentioned this issue Nov 10, 2015
Closed
@Gigitsu
Copy link

Gigitsu commented Mar 3, 2016

How is this supposed to work? I'm having problem with typescript, coverage preprocessor strips away every sourceMappingURL comment from my js files

@vsakaria
Copy link

I am having problems to with karma-coverage and sourcemaps used in grunt-karma. Any idea's??

MichaReiser pushed a commit to MichaReiser/istanbul-instrumenter-loader that referenced this issue Sep 6, 2016
Pass intermediate source map to instrumenter
cb44c62 
Pass the intermediate source map to the instrumenter that can be used
to remap the coverage report to the original source.
Requires istanbuljs-archived-repos/istanbul-lib-instrument#23.

Fixes webpack-contrib#22, webpack-contrib#7, karma-runner/karma-coverage#109,
@MichaReiser MichaReiser mentioned this issue Sep 6, 2016
Merged
@MichaReiser MichaReiser mentioned this issue Sep 14, 2016
Merged
deepsweet pushed a commit to webpack-contrib/istanbul-instrumenter-loader that referenced this issue Nov 27, 2016
➕ add SourceMaps remapping support (close #22, close #25)
d49b054 
* Upgrade to istanbul-lib-instrument

* Pass intermediate source map to instrumenter

Pass the intermediate source map to the instrumenter that can be used
to remap the coverage report to the original source.
Requires istanbuljs-archived-repos/istanbul-lib-instrument#23.

Fixes #22, #7, karma-runner/karma-coverage#109,
This was referenced Feb 22, 2020
Open
Open
Open
Open
Open
Open
Open
Open
This was referenced Feb 23, 2020
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
PR exists
Projects
None yet
Milestone
No milestone
7 participants
@mwisnicki @dotnetwise @dignifiedquire @dayyan @vsakaria @Gigitsu @GoulwenBZH