covr icon indicating copy to clipboard operation
covr copied to clipboard

Updating Codecov script to mirror bash script

Open thomasrockhu opened this issue 3 years ago • 2 comments

Hi, Tom from Codecov here. I noticed an uptick in our R code coverage, and I wanted to introduce myself and see what I can do to help. I am not an R developer nor am I too familiar with the ecosystem, so you'll have to forgive me for any mistakes.

Looking at the code, I notice two improvements that could be made

  1. uploading using the v4 endpoint as opposed to v2 and
  2. adding more CI/CDs to CI detection. I am happy to write a PR if that is helpful.

Although we do offer other community uploaders like our Ruby uploader, we have been trying to move towards wrappers on top of our bash uploader like our Action to ensure the latest and greatest. Would this be interesting to you at all?

thomasrockhu avatar Mar 08 '21 23:03 thomasrockhu

Sure, we can move to using the v4 endpoint. I am not sure we need to worry about adding more CI services, what we have seem sufficient for the vast majority of cases, do you have a list of which CI services you would add?

jimhester avatar Mar 12 '21 18:03 jimhester

https://about.codecov.io/security-update/

I'm sure glad we are using covr instead of the bash uploader latest and greatest.

meztez avatar Apr 15 '21 13:04 meztez