Skip to content

Travis: coverage: gcov: process .gnco separately#2574

Merged
blueyed merged 6 commits intoawesomeWM:masterfrom
blueyed:gcov
Jan 10, 2019
Merged

Travis: coverage: gcov: process .gnco separately#2574
blueyed merged 6 commits intoawesomeWM:masterfrom
blueyed:gcov

Conversation

@blueyed
Copy link
Copy Markdown
Member

@blueyed blueyed commented Jan 10, 2019

Taken out of #2564.

It is likely a fix for the flaky coverage with C files we're seeing currently.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 10, 2019

Codecov Report

Merging #2574 into master will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2574      +/-   ##
==========================================
+ Coverage   84.57%   84.65%   +0.08%     
==========================================
  Files         497      496       -1     
  Lines       33729    33738       +9     
==========================================
+ Hits        28526    28562      +36     
+ Misses       5203     5176      -27
Flag Coverage Δ
#c_code 73.53% <ø> (+0.41%) ⬆️
#functionaltests 70.96% <ø> (ø) ⬆️
#lua53 84.65% <ø> (-2.98%) ⬇️
#samples 74.65% <ø> (ø) ⬆️
#themes 49.77% <ø> (ø) ⬆️
#unittests 58.17% <ø> (ø) ⬆️
Impacted Files Coverage Δ
build-utils/lgi-check.c
luaa.h 67.46% <0%> (+0.8%) ⬆️
common/util.h 89.28% <0%> (+3.57%) ⬆️
common/buffer.h 75.75% <0%> (+75.75%) ⬆️
event.h 100% <0%> (+100%) ⬆️

@blueyed blueyed merged commit 3804040 into awesomeWM:master Jan 10, 2019
@blueyed blueyed deleted the gcov branch January 10, 2019 22:07
petoju pushed a commit to petoju/awesome that referenced this pull request Jun 8, 2019
Also uses consistent flags with codecov, i.e. lua53 also with gcov / `#c_code`.
actionless pushed a commit to actionless/awesome that referenced this pull request Mar 24, 2021
Also uses consistent flags with codecov, i.e. lua53 also with gcov / `#c_code`.
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.

1 participant