Skip to content

Conversation

@pitrou
Copy link
Member

@pitrou pitrou commented Nov 5, 2019

No description provided.

after_success:
- pushd ${TRAVIS_BUILD_DIR}/go/arrow
- bash <(curl -s https://codecov.io/bash) || echo "Codecov did not collect coverage reports"
#- bash <(curl -s https://codecov.io/bash) || echo "Codecov did not collect coverage reports"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sbinet Do you use the coverage reports? Otherwise, is there something more to do to disable Go coverage?

after_success:
- Rscript ../ci/travis_upload_r_coverage.R
#after_success:
#- Rscript ../ci/travis_upload_r_coverage.R
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nealrichardson Is there something more to do to disable R coverage computation?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions
Copy link

github-actions bot commented Nov 5, 2019

@pitrou
Copy link
Member Author

pitrou commented Nov 6, 2019

I didn't get any objection on the ML, so will merge. This can be easily re-enabled anyway.

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.

2 participants