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 Github status to PR for c2go tests and warnings #338

Merged
merged 5 commits into from
Nov 9, 2017

Conversation

elliotchance
Copy link
Owner

@elliotchance elliotchance commented Nov 8, 2017

This change is Reviewable

@codecov
Copy link

codecov bot commented Nov 8, 2017

Codecov Report

Merging #338 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #338   +/-   ##
======================================
  Coverage    81.6%   81.6%           
======================================
  Files         142     142           
  Lines        6011    6011           
======================================
  Hits         4905    4905           
  Misses        886     886           
  Partials      220     220

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 edf5fee...3880987. Read the comment docs.

@elliotchance
Copy link
Owner Author

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@elliotchance elliotchance merged commit 7a77f63 into master Nov 9, 2017
@elliotchance elliotchance deleted the github-status branch November 9, 2017 03:01
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