Skip to content

Add GHA batch builds which use fetch content#2545

Merged
blowekamp merged 1 commit intoSimpleITK:mainfrom
blowekamp:gha_batch_use_fetchcontent
Mar 27, 2026
Merged

Add GHA batch builds which use fetch content#2545
blowekamp merged 1 commit intoSimpleITK:mainfrom
blowekamp:gha_batch_use_fetchcontent

Conversation

@blowekamp
Copy link
Copy Markdown
Member

No description provided.

@blowekamp blowekamp force-pushed the gha_batch_use_fetchcontent branch 3 times, most recently from b32c2fd to 0774d0d Compare March 27, 2026 13:27
@blowekamp blowekamp marked this pull request as ready for review March 27, 2026 13:27
@blowekamp blowekamp requested a review from Copilot March 27, 2026 13:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the BatchBuild GitHub Actions workflow to support running certain matrix jobs without the SuperBuild, aligning the CTest invocation with other workflows by passing the dashboard source config directory.

Changes:

  • Add use-superbuild: false to selected matrix entries to indicate non-SuperBuild builds.
  • Conditionally install additional Python build dependencies and inject SWIG/Python cache entries for non-SuperBuild jobs.
  • Pass dashboard_source_config_dir into the ctest dashboard script based on use-superbuild.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/BatchBuild.yml Outdated
Comment thread .github/workflows/BatchBuild.yml
@blowekamp blowekamp force-pushed the gha_batch_use_fetchcontent branch from 0774d0d to 9a10d6e Compare March 27, 2026 13:54
@blowekamp blowekamp force-pushed the gha_batch_use_fetchcontent branch from 9a10d6e to 8c1e638 Compare March 27, 2026 14:49
@blowekamp blowekamp requested review from dave3d and zivy March 27, 2026 18:27
@blowekamp blowekamp merged commit eddb856 into SimpleITK:main Mar 27, 2026
16 checks passed
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.

4 participants