Skip to content

Further fixes to CI - #172

Merged
dzenanz merged 3 commits into
InsightSoftwareConsortium:mainfrom
dzenanz:greenCI
Oct 2, 2026
Merged

dzenanz merged 3 commits into
InsightSoftwareConsortium:mainfrom
dzenanz:greenCI

Conversation

@dzenanz

@dzenanz dzenanz commented Oct 1, 2026

Copy link
Copy Markdown
Member

This set of changes made CI green in my fork.

@dzenanz
dzenanz requested review from hjmjohnson and thewtex and a balanced review from Copilot October 1, 2026 14:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Cross-platform packaging depends on external ITK 6 artifacts, and the live CI matrix has not completed.

Review effort: Balanced
Findings: None

What changed in this PR

Updates CI for ITK 6 artifacts and Python 3.11+ packaging.

Changes:

  • Migrates Windows jobs to Visual Studio 2026 runners.
  • Builds Python 3.11 wheels using available platform archives.
  • Raises generated packages’ minimum Python version to 3.11.
File Description
.github/​workflows/​build-test-package.yml Updates CI runners and packaging matrices.
{{cookiecutter.project_name}}/​pyproject.toml Requires Python 3.11+.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@dzenanz

dzenanz commented Oct 1, 2026

Copy link
Copy Markdown
Member Author

@greptileai review this.

@dzenanz

dzenanz commented Oct 1, 2026

Copy link
Copy Markdown
Member Author

Greptile isn't enabled in this repo. I think this is good to merge. It turns the CI green.

@thewtex thewtex left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@dzenanz thanks!

@dzenanz
dzenanz merged commit c1e0bbd into InsightSoftwareConsortium:main Oct 2, 2026
7 checks passed
@dzenanz
dzenanz deleted the greenCI branch October 2, 2026 01:09
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.

3 participants