Skip to content

add codecov to GH Actions#130

Merged
cprivitere merged 3 commits intomasterfrom
coveragev2
Jan 6, 2022
Merged

add codecov to GH Actions#130
cprivitere merged 3 commits intomasterfrom
coveragev2

Conversation

@displague
Copy link
Member

@displague displague commented Jan 5, 2022

Adds code coverage reporting to GH Actions

@displague displague force-pushed the coveragev2 branch 3 times, most recently from 654fd42 to 42c08f0 Compare January 5, 2022 23:01
Signed-off-by: Marques Johansson <mjohansson@equinix.com>
@codecov
Copy link

codecov bot commented Jan 5, 2022

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #130   +/-   ##
=========================================
  Coverage          ?   60.79%           
=========================================
  Files             ?       23           
  Lines             ?      987           
  Branches          ?       95           
=========================================
  Hits              ?      600           
  Misses            ?      376           
  Partials          ?       11           
Flag Coverage Δ
unittests 60.79% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 4f307ff...953b86d. Read the comment docs.

Signed-off-by: Marques Johansson <mjohansson@equinix.com>
@displague
Copy link
Member Author

AFAICT, the links in the graph won't work until the first upload is made on the master branch
https://app.codecov.io/gh/packethost/packet-python/

@cprivitere
Copy link

Should your envlist still have the python versions you removed? "envlist = py27,py36,py37,py38,py39,py310"

Signed-off-by: Marques Johansson <mjohansson@equinix.com>
@displague displague requested a review from cprivitere January 6, 2022 17:56
@cprivitere cprivitere merged commit 8c4fe6f into master Jan 6, 2022
@cprivitere cprivitere deleted the coveragev2 branch January 6, 2022 18:21
@displague displague restored the coveragev2 branch June 10, 2022 13:12
@displague displague deleted the coveragev2 branch July 27, 2022 14:00
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.

2 participants