feat: add r-hub workflow #232
Conversation
|
Code Coverage SummaryDiff against mainResults for commit: cc4fc0f Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary3 tests 3 ✅ 6s ⏱️ Results for commit cc4fc0f. ♻️ This comment has been updated with latest results. |
…r.pkg.template into test-rhub-workflows
|
This is pending on insightsengineering/setup-r-dependencies#1. |
…r.pkg.template into test-rhub-workflows
Signed-off-by: walkowif <[email protected]>
Co-authored-by: Pawel Rucki <[email protected]> Signed-off-by: walkowif <[email protected]>
Signed-off-by: walkowif <[email protected]>
.github/workflows/rhub.yaml
Outdated
| # All supported R-hub platforms can be viewed by running: rhub::rhub_platforms() | ||
| # Container-based platforms (as opposed to VM-based Windows and macOS platforms) | ||
| # can be also viewed here: https://r-hub.github.io/containers/containers.html | ||
| default: 'ubuntu-clang,ubuntu-gcc12,gcc13,ubuntu-next,ubuntu-release,macos-arm64,macos,windows,gcc14,atlas,valgrind,intel,nosuggests,mkl' |
There was a problem hiding this comment.
Opening a separate thread to keep the discussion well organised.
This is the results for teal:
Take a look at the timings for valgrind. It took 4 times longer than the median timing of all the platforms. If the minutes consumption are in concern - maybe we can remove it? WDYT?
There was a problem hiding this comment.
AFAIK, GitHub minutes consumption is not a problem for public repositories (they are free) but we can remove any irrelevant checks, especially if they take a long time.
Co-authored-by: Pawel Rucki <[email protected]> Signed-off-by: walkowif <[email protected]>
https://github.com/insightsengineering/idr-tasks/issues/781