search

manifoldco / promptui

Interactive prompt for command-line applications
https://www.manifold.co
BSD 3-Clause "New" or "Revised" License
6.03k stars 333 forks source link

add coverage reporting #90

Closed davidharrigan closed 5 years ago

davidharrigan commented 5 years ago

This PR adds coverage reporting to promptui. Since promptui runs tests run for Go versions 1.9, 1.10, and 1.11 - I've configured it to only publish the report to codecov for 1.11.

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.

enmand commented 5 years ago

@jbowes, @PLaRoche or someone with admin on the org might need to add/enable CodeCode for this repo

jbowes commented 5 years ago

@jbowes, @PLaRoche or someone with admin on the org might need to add/enable CodeCode for this repo

all done!

codecov[bot] commented 5 years ago

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #90   +/-   ##
=========================================
  Coverage          ?   31.25%           
=========================================
  Files             ?        6           
  Lines             ?      659           
  Branches          ?        0           
=========================================
  Hits              ?      206           
  Misses            ?      433           
  Partials          ?       20

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 06dc694...e106bc0. Read the comment docs.

davidharrigan commented 5 years ago

@enmand - I added a new commit to collect coverage for each package separately as the 1.9 version of the build was failing because of that. Let me know if that looks reasonable!