Skip to content

Ci/gha test fresh pr - #697

Closed
timcadman wants to merge 66 commits into
v7.0-devfrom
ci/gha-test-fresh-pr
Closed

Ci/gha test fresh pr#697
timcadman wants to merge 66 commits into
v7.0-devfrom
ci/gha-test-fresh-pr

Conversation

@timcadman

@timcadman timcadman commented Sep 1, 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 September 1, 2026 14:50
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
⏳ 2 of 5 checks reported so far
CheckResult
Devtools checks✅ passed
Code quality✅ 0 findings
Armadillo unit tests⏳ pending
Opal unit tests⏳ pending
Test coverage⏳ pending

Tested against dsBase versions:
Armadillo: pending
Opal: pending

Logs: Devtools checks · Code quality · pending · pending · pending

@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.

1 participant