JuliaCI / Coverage.jl

Take Julia code coverage and memory allocation results, do useful things with them

MIT License

174 stars 68 forks source link

Expand on data extraction from github action runs for Coveralls #296

Closed mfherbst closed 3 years ago

mfherbst commented 3 years ago

Based on the official docs.

mfherbst commented 3 years ago

Regarding tests ... it's not quite clear to me how to cover this. Happy for ideas.

codecov-io commented 3 years ago

Codecov Report

Merging #296 (a142970) into master (36fc28a) will increase coverage by 0.72%. The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #296      +/-   ##
==========================================
+ Coverage   81.90%   82.63%   +0.72%     
==========================================
  Files           3        3              
  Lines         199      190       -9     
==========================================
- Hits          163      157       -6     
+ Misses         36       33       -3

Impacted Files	Coverage Δ
src/codecovio.jl	`76.74% <0.00%> (+0.65%)`	:arrow_up:
src/coveralls.jl	`87.37% <66.66%> (+0.58%)`	:arrow_up:

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 36fc28a...a142970. Read the comment docs.

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 941

1 of 4 (25.0%) changed or added relevant lines in 2 files are covered.
10 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.3%) to 83.108%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/coveralls.jl	1	2	50.0%
src/codecovio.jl	0	2	0.0%
<!--	Total:	1	4	25.0%	-->

Files with Coverage Reduction	New Missed Lines	%
src/codecovio.jl	1	63.95%
src/coveralls.jl	9	66.02%
<!--	Total:	10	-->

Totals
Change from base Build 932:	0.3%
Covered Lines:	123
Relevant Lines:	148

💛 - Coveralls

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 937

0 of 4 (0.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.06%) to 80.745%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/coveralls.jl	0	4	0.0%
<!--	Total:	0	4	0.0%	-->

Totals
Change from base Build 932:	-2.06%
Covered Lines:	130
Relevant Lines:	161

💛 - Coveralls

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 937

0 of 4 (0.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.06%) to 80.745%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/coveralls.jl	0	4	0.0%
<!--	Total:	0	4	0.0%	-->

Totals
Change from base Build 932:	-2.06%
Covered Lines:	130
Relevant Lines:	161

💛 - Coveralls

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 937

0 of 4 (0.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.06%) to 80.745%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/coveralls.jl	0	4	0.0%
<!--	Total:	0	4	0.0%	-->

Totals
Change from base Build 932:	-2.06%
Covered Lines:	130
Relevant Lines:	161

💛 - Coveralls

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 937

0 of 4 (0.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.06%) to 80.745%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/coveralls.jl	0	4	0.0%
<!--	Total:	0	4	0.0%	-->

Totals
Change from base Build 932:	-2.06%
Covered Lines:	130
Relevant Lines:	161

💛 - Coveralls

mfherbst commented 3 years ago

:+1: Thanks for merging.