gofrs / uuid

A UUID package for Go
MIT License
1.58k stars 113 forks source link

build: replace coveralls with codecov #56

Closed adamdecaf closed 6 years ago

adamdecaf commented 6 years ago

Fixes: https://github.com/gofrs/uuid/issues/55

codecov-io commented 6 years ago

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #56   +/-   ##
=========================================
  Coverage          ?   98.91%           
=========================================
  Files             ?        4           
  Lines             ?      276           
  Branches          ?        0           
=========================================
  Hits              ?      273           
  Misses            ?        2           
  Partials          ?        1

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 47cd1dc...18b0ced. Read the comment docs.

theckman commented 6 years ago

@acln0 going to venture to say he used the Travis command line tool to configure it.

acln0 commented 6 years ago

Ah okay. Well, I don't think this is a big deal anyway. I was asking out of curiosity.

acln0 commented 6 years ago

The readme still links to coveralls, though. Let's change it after this gets merged. Thanks @adamdecaf!

adamdecaf commented 6 years ago

Ah. It was emacs. I'll update the readme.

adamdecaf commented 6 years ago

A preview: https://github.com/adamdecaf/uuid/tree/codecov#uuid

acln0 commented 6 years ago

Thanks. Can you please squash these down to one commit and push again?

adamdecaf commented 6 years ago

Let's hope codecov rescans.

acln0 commented 6 years ago

That's a strange failure. EDIT: never mind, it's re-running it, apparently.

adamdecaf commented 6 years ago

That's me. The error was just in go get for cover. Not our fault.

acln0 commented 6 years ago

Thanks!