-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Various fixes on GitHub Actions CI #3933
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
Various fixes on GitHub Actions CI #3933
Conversation
@clauswilke @thomasp85 |
.github/workflows/R-CMD-check.yaml
Outdated
# if: matrix.config.os == 'macOS-latest' && matrix.config.r == '3.6' | ||
# run: covr::codecov() | ||
# shell: Rscript {0} | ||
- name: Test coverage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be worthwhile to remove in favour of a separate action (that's what we're using elsewhere)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, sounds nice! I'll remove this step and add https://github.com/r-lib/actions/blob/master/.github/workflows/test-coverage.yaml.
This PR addresses the following issues:
Add a workaround for problem about Rcpp on macOS runner (Many packages are broken on Mac with CRAN version of Rcpp -- unless 10.11 SDK is used as on CRAN RcppCore/Rcpp#1060).(Rcpp is fixed on CRAN)cache-version
to enable cache clear.