search

kevlened / pytest-parallel

A pytest plugin for parallel and concurrent testing
https://github.com/browsertron/pytest-parallel/issues/104#issuecomment-1293941066
MIT License
313 stars 60 forks source link

[WIP] ci: report coverage via codecov #57

Closed blueyed closed 4 years ago

blueyed commented 4 years ago

Fixes https://github.com/browsertron/pytest-parallel/issues/56.

blueyed commented 4 years ago

Hmm, Travis created two "Travis CI - Pull Request" builds?

@kevlened Did you change some config in this regard?

codecov-io commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@75c8bc3). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #57   +/-   ##
=========================================
  Coverage          ?   70.86%           
=========================================
  Files             ?        5           
  Lines             ?      357           
  Branches          ?       35           
=========================================
  Hits              ?      253           
  Misses            ?       93           
  Partials          ?       11
Flag Coverage Δ
#linux 70.86% <ø> (?)
#py36 70.86% <ø> (?)
#py37 70.86% <ø> (?)

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 75c8bc3...4a83816. Read the comment docs.

kevlened commented 4 years ago

I haven't changed anything. I'm not sure how that happened.

blueyed commented 4 years ago

ok, fine with a rebuild, so likely a hiccup on their end.

blueyed commented 4 years ago

@kevlened Thanks for merging, but I wanted to look at it more closely (therefore "[WIP]"). Anyway, not too bad - and at least we have a base report then to follow up upon.. :)

kevlened commented 4 years ago

Sorry, jumped the gun. Just excited.

blueyed commented 4 years ago

Sorry, jumped the gun. Just excited.

Fair enough.. :)

blueyed commented 4 years ago

I'm glad you're doing squash-merges in the first place, but you can also edit them (e.g. to not include "[WIP]" or "rebuild" in there). JFI, thanks for being pro-active!