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.

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

Add support for Sourcelink #260

Merged
merged 5 commits into from
Dec 9, 2018
Merged

Add support for Sourcelink #260

merged 5 commits into from
Dec 9, 2018

Conversation

tonerdo
Copy link
Collaborator

@tonerdo tonerdo commented Dec 9, 2018

You can now specify that Coverlet uses SourceLink URIs in place of document paths in coverage results.

MSBuild:

dotnet test /p:CollectCoverage=true /p:UseSourceLink=true

Global Tool:

coverlet test-assembly.dll --target "dotnet" --targetargs "test test-project --no-build" --use-source-link

cc @ViktorHofer

Fixes #257

@codecov
Copy link

codecov bot commented Dec 9, 2018

Codecov Report

Merging #260 into master will decrease coverage by 2%.
The diff coverage is 35.08%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
- Coverage   95.44%   93.43%   -2.01%     
==========================================
  Files          16       16              
  Lines        1757     1783      +26     
==========================================
- Hits         1677     1666      -11     
- Misses         80      117      +37
Impacted Files Coverage Δ
...overlet.core/Instrumentation/InstrumenterResult.cs 100% <ø> (ø) ⬆️
src/coverlet.core/Reporters/CoberturaReporter.cs 98.23% <100%> (-0.33%) ⬇️
src/coverlet.core/Instrumentation/Instrumenter.cs 98.95% <100%> (+0.01%) ⬆️
src/coverlet.core/Coverage.cs 85.15% <26%> (-14.37%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ead796...1b2d8ba. Read the comment docs.

@tonerdo tonerdo merged commit b819588 into master Dec 9, 2018
@ViktorHofer
Copy link
Contributor

You know that question was coming. When can we consume that change?

@tonerdo
Copy link
Collaborator Author

tonerdo commented Dec 10, 2018

@ViktorHofer most likely tomorrow. I need to add a few more things before making a new release

@tonerdo tonerdo deleted the sourcelink branch December 17, 2018 09:10
@tonerdo
Copy link
Collaborator Author

tonerdo commented Dec 20, 2018

@ViktorHofer sorry it took much longer than expected, here's the new release with source link support https://www.nuget.org/packages/coverlet.console/1.4.0

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.

2 participants