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

[WIP] ci: try lcov #2732

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[WIP] ci: try lcov #2732

wants to merge 4 commits into from

Conversation

blueyed
Copy link
Member

@blueyed blueyed commented Mar 24, 2019

No description provided.

@blueyed
Copy link
Member Author

blueyed commented Mar 24, 2019

Should get integrated into #2672, if it turns out to work better.
But #2672 could be merged already.

@codecov
Copy link

codecov bot commented Mar 24, 2019

Codecov Report

Merging #2732 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2732      +/-   ##
==========================================
- Coverage    86.5%   86.49%   -0.01%     
==========================================
  Files         569      570       +1     
  Lines       38076    38088      +12     
==========================================
+ Hits        32937    32945       +8     
- Misses       5139     5143       +4
Flag Coverage Δ
#gcov 74.96% <ø> (-0.02%) ⬇️
#luacov 89.39% <ø> (ø) ⬆️
Impacted Files Coverage Δ
build-utils/lgi-check.c 66.66% <0%> (ø)

@blueyed
Copy link
Member Author

blueyed commented Mar 24, 2019

Misses a bit (including spawn_monitor_timeout and spawn_launchee_timeout) - is that only flaky?

@blueyed
Copy link
Member Author

blueyed commented Mar 24, 2019

NOTE: many parenthesis related changes when skipping fixes with codecov (-X fix): https://codecov.io/gh/awesomeWM/awesome/compare/ed0c3ceb2c39d0799cfe86894ad9dc2ff5e0e40c...0113666e83a5488a3a4e8cb6ced341869498335f/changes

@blueyed blueyed closed this Apr 3, 2019
@blueyed blueyed reopened this Apr 3, 2019
@blueyed
Copy link
Member Author

blueyed commented Apr 3, 2019

Re-trigger coveralls.

@blueyed blueyed changed the title ci: try lcov [WIP] ci: try lcov Jun 15, 2019
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.

None yet

1 participant