Skip to content

Fix continue-on-error in composite actions to use boolean expression#2557

Merged
blowekamp merged 1 commit intoSimpleITK:mainfrom
blowekamp:gha_actions_continue
Apr 11, 2026
Merged

Fix continue-on-error in composite actions to use boolean expression#2557
blowekamp merged 1 commit intoSimpleITK:mainfrom
blowekamp:gha_actions_continue

Conversation

@blowekamp
Copy link
Copy Markdown
Member

No description provided.

@blowekamp blowekamp marked this pull request as ready for review April 10, 2026 21:00
@blowekamp blowekamp requested a review from Copilot April 10, 2026 21:00
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

Updates composite GitHub Actions so continue-on-error evaluates as a proper boolean (instead of a truthy/falsey string), preventing "false" from being treated as enabled.

Changes:

  • Set continue-on-error input defaults to the string 'false' where needed.
  • Update continue-on-error usage to a boolean expression: ${{ inputs.continue-on-error == 'true' }} across Python/Java/CSharp packaging actions.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/actions/package_python/action.yml Uses boolean expression for continue-on-error and makes its default a string.
.github/actions/package_java/action.yml Adds string default for continue-on-error and compares to 'true' to produce a boolean.
.github/actions/package_csharp/action.yml Adds string default for continue-on-error and compares to 'true' to produce a boolean.

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

@blowekamp blowekamp merged commit c26c878 into SimpleITK:main Apr 11, 2026
7 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.

2 participants