pydoit / doit

CLI task management & automation tool

http://pydoit.org

MIT License

1.85k stars 175 forks source link

add codecov, tweak some spelling #384

Closed bollwyvl closed 3 years ago

bollwyvl commented 3 years ago

A follow-up to https://github.com/pydoit/doit/pull/379#pullrequestreview-611689406, this uploads coverage to codecov, which is slightly simpler to set up, and doesn't require a secret token.

Also fixes a few spelling issues, just so we can see all the :heavy_check_mark: .

bollwyvl commented 3 years ago

I think to get the report to show up in the issue, one button must be pushed, once, in the codecov app.

coveralls commented 3 years ago

Coverage remained the same at 99.729% when pulling 9ea3645d4cb7df2be762127e9835728448d0bc8a on bollwyvl:add-codecov into 0aa4cc55fdd90b4f4e3480fb3ca089be572053c5 on pydoit:master.

codecov-io commented 3 years ago

Codecov Report

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

@@            Coverage Diff            @@
##             master     #384   +/-   ##
=========================================
  Coverage          ?   99.72%           
=========================================
  Files             ?       65           
  Lines             ?     8828           
  Branches          ?        0           
=========================================
  Hits              ?     8804           
  Misses            ?       24           
  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 0aa4cc5...9ea3645. Read the comment docs.

bollwyvl commented 3 years ago

Adding the codecov badge does require a (non-secret) token... the markdown/rst can be copied from the settings page on codecov.

schettino72 commented 3 years ago

  .. image:: https://codecov.io/gh/pydoit/doit/branch/master/graph/badge.svg?token=wxKa1h11zn
  :target: https://codecov.io/gh/pydoit/doit

schettino72 commented 3 years ago

When I click on "continue to full report" [1], it says the page is missing.

[1] https://app.codecov.io/gh/pydoit/doit/compare/384

bollwyvl commented 3 years ago

Yeah, once some reports hit master, it starts working more.

bollwyvl commented 3 years ago

oh, merged! welp, added some badges on my push, feel free to steal!

schettino72 commented 3 years ago

merged. thanks.