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

Run coverage in parallel mode #5445

Merged
merged 12 commits into from Apr 14, 2017

Conversation

Projects
None yet
1 participant
@benjaoming
Member

benjaoming commented Apr 14, 2017

Summary

Been wondering a bit why coverage isn't showing up properly for kalite.cli so trying out this, even though I think CodeCov already knows that it's receiving separate reports from both CircleCI instances...

Supersedes #5443

@benjaoming benjaoming added this to the 0.17.1 milestone Apr 14, 2017

@benjaoming benjaoming self-assigned this Apr 14, 2017

@benjaoming benjaoming added the has PR label Apr 14, 2017

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Apr 14, 2017

Codecov Report

Merging #5445 into 0.17.x will increase coverage by 9.45%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           0.17.x    #5445      +/-   ##
==========================================
+ Coverage   51.94%   61.39%   +9.45%     
==========================================
  Files         143      118      -25     
  Lines        7466     6574     -892     
==========================================
+ Hits         3878     4036     +158     
+ Misses       3588     2538    -1050
Impacted Files Coverage Δ
kalite/coachreports/api_urls.py 100% <0%> (ø) ⬆️
kalite/contentload/api_urls.py 100% <0%> (ø) ⬆️
kalite/updates/urls.py 100% <0%> (ø) ⬆️
kalite/inline/api_urls.py 100% <0%> (ø) ⬆️
kalite/distributed/forms.py 100% <0%> (ø) ⬆️
kalite/control_panel/urls.py 100% <0%> (ø) ⬆️
kalite/inline/narratives.py 100% <0%> (ø) ⬆️
...lite/control_panel/features/steps/control_panel.py
kalite/project/settings/base.py
kalite/distributed/features/steps/generic.py
... and 58 more

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 062188a...f29a8b5. Read the comment docs.

codecov bot commented Apr 14, 2017

Codecov Report

Merging #5445 into 0.17.x will increase coverage by 9.45%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           0.17.x    #5445      +/-   ##
==========================================
+ Coverage   51.94%   61.39%   +9.45%     
==========================================
  Files         143      118      -25     
  Lines        7466     6574     -892     
==========================================
+ Hits         3878     4036     +158     
+ Misses       3588     2538    -1050
Impacted Files Coverage Δ
kalite/coachreports/api_urls.py 100% <0%> (ø) ⬆️
kalite/contentload/api_urls.py 100% <0%> (ø) ⬆️
kalite/updates/urls.py 100% <0%> (ø) ⬆️
kalite/inline/api_urls.py 100% <0%> (ø) ⬆️
kalite/distributed/forms.py 100% <0%> (ø) ⬆️
kalite/control_panel/urls.py 100% <0%> (ø) ⬆️
kalite/inline/narratives.py 100% <0%> (ø) ⬆️
...lite/control_panel/features/steps/control_panel.py
kalite/project/settings/base.py
kalite/distributed/features/steps/generic.py
... and 58 more

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 062188a...f29a8b5. Read the comment docs.

@benjaoming benjaoming merged commit 150cb9b into learningequality:0.17.x Apr 14, 2017

@benjaoming benjaoming removed the has PR label Apr 14, 2017

@benjaoming benjaoming deleted the benjaoming:coverage-parallel2 branch Apr 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment