Skip to content

Support Google Cloud Build #469

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jun 7, 2022

Conversation

MarkEdmondson1234
Copy link
Contributor

This lets googleCloudRunner upload covr reports directly without needing travis as well.

I referenced the API arguments here: https://docs.codecov.io/reference#upload

I enabled the default substitution variables available in Google Cloud Build here: https://cloud.google.com/build/docs/configuring-builds/substitute-variable-values - creating them as environment arguments for covr to discover with the prefix GCB_

I have some successful Cloud Build reports uploading with this code here: https://app.codecov.io/gh/MarkEdmondson1234/measurementProtocol/

@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
All committers have signed the CLA.

@jimhester
Copy link
Member

Thanks, this looks good!

Can you please add a bullet to NEWS? It should briefly describe the change and end with (@yourname, #issuenumber).

@MarkEdmondson1234
Copy link
Contributor Author

Ah thanks! Its been a while so had to remind myself, but I've been using my fork extensively for example during development of https://app.codecov.io/gh/MarkEdmondson1234/googleCloudRunner

@jimhester
Copy link
Member

Sorry about the delay, I think I was not actively watching this repo when you sent the PR, so didn't see it right away.

@jimhester jimhester merged commit 1360330 into r-lib:main Jun 7, 2022
@jimhester
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants