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

Start running test suite with GitHub actions and reporting coverage to Codecov #17

Closed
wants to merge 7 commits into from
Closed

Start running test suite with GitHub actions and reporting coverage to Codecov #17

wants to merge 7 commits into from

Conversation

etagwerker
Copy link
Member

Hi there,

This is an attempt to run the test suite on every PR and report code coverage metrics to Codecov.

Please check it out.

Thanks!

This will be necessary to keep track of code coverage metrics
This will be needed to report data to CodeCov and to the console
@etagwerker etagwerker requested a review from bleonard June 12, 2021 01:47
@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@638b081). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #17   +/-   ##
=========================================
  Coverage          ?   69.68%           
=========================================
  Files             ?       14           
  Lines             ?      597           
  Branches          ?        0           
=========================================
  Hits              ?      416           
  Misses            ?      181           
  Partials          ?        0           

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 638b081...1659d12. Read the comment docs.

@kindoflew kindoflew changed the base branch from master to main September 8, 2021 19:24
@lubc lubc requested review from rdormer and removed request for bleonard September 10, 2021 17:06
@lubc
Copy link
Contributor

lubc commented Sep 10, 2021

@rdormer hey, could you review this PR when you have some time? Thanks!

@bronzdoc
Copy link
Contributor

@etagwerker it is expected that the test suite fails in all rubies?

@etagwerker
Copy link
Member Author

@bronzdoc Weird, that is unexpected. I'll look into it.

@etagwerker
Copy link
Member Author

@lubc @bronzdoc Closing this in favor of #18

@etagwerker etagwerker closed this Sep 22, 2021
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.

5 participants