Skip to content

Ci/gha parallel test suite - #693

Closed
timcadman wants to merge 60 commits into
v7.0-devfrom
ci/gha-parallel-test-suite
Closed

Ci/gha parallel test suite#693
timcadman wants to merge 60 commits into
v7.0-devfrom
ci/gha-parallel-test-suite

Conversation

@timcadman

@timcadman timcadman commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Background

The existing Azure CI has worked well, but it can potentially be improved.

What's changed

  • Migrated from Azure to github actions

New features

  • Runs Opal and Armadillo each time
  • Tests are run in parallel batches reducing time from >1 hour to <20 mins
  • Added lint testing for code quality
  • Add a summary of passes/fails in the comments

Still to do

  • I have also tried to add code cov but I can't get it working - maybe we look at this together.

Testing

This was very AI generated flow, but I have asked it to review its own code and it assures me that none of the existing functionality is lost. I have also tested it against different combinations of passing and failing.

@timcadman
timcadman changed the base branch from v6.3.6-dev to v7.0-dev August 25, 2026 09:04
@github-actions

Copy link
Copy Markdown

✅ Package checks

Check Result
Doc sync (man/*.Rd vs R headers) ✅ passed
R CMD check (0 errors/warnings/notes) ✅ passed

View full log

@github-actions

Copy link
Copy Markdown

✅ Lint

@github-actions

Copy link
Copy Markdown

✅ Tests

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
❌ 4 of 5 checks failed

Tested against dsBase versions:
Armadillo: 7.0.0.9000
Opal: 7.0.0.9000

CheckResult
Devtools checks❌ failed
Code quality❌ 2 findings
Armadillo unit tests[ FAIL 0 | WARN 0 | SKIP 27 | PASS 11026 ]
Opal unit tests[ FAIL 0 | WARN 0 | SKIP 10 | PASS 8147 ]
Test coverage❌ 61.3% vs 80% target

Logs: Devtools checks · Code quality · Armadillo unit tests · Opal unit tests · Codecov

Comment thread R/checkClass.R
#'
checkClass <- function(datasources=NULL, obj=NULL){
# check the class of the input object
verbose <- T
Comment thread R/checkClass.R
#'
checkClass <- function(datasources=NULL, obj=NULL){
# check the class of the input object
verbose <- T
@timcadman timcadman closed this Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants