Skip to content

Workflow adjustments#2301

Merged
Zeitsperre merged 5 commits intomainfrom
workflow-adjustments
Jan 9, 2026
Merged

Workflow adjustments#2301
Zeitsperre merged 5 commits intomainfrom
workflow-adjustments

Conversation

@Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Jan 9, 2026

What kind of change does this PR introduce?

  • Sets an output condition to prevent subsequent steps from running (without marking the workflow as cancelled/failed).
  • Addresses a concurrency issue that occurs when a pull_request_review event cancels a pull_request triggered build.
  • Completely removes ubuntu-slim (not compatible with step-security/harden-runner).

Does this PR introduce a breaking change?

Pull request reviews will no longer cancel workflows that have been trigger via pull_request events (synchronize). This should make things a bit easier.

Other information:

Given that this workflow performs a checkout and creates tokens using secrets, we should not be allowing pull_request_target as an allowed trigger.

Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
@Zeitsperre Zeitsperre self-assigned this Jan 9, 2026
@github-actions github-actions bot added the CI Automation and Contiunous Integration label Jan 9, 2026
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

Note

It appears that this Pull Request modifies the main.yml workflow.

On inspection, the XCLIM_TESTDATA_BRANCH environment variable is set to the most recent tag (v2025.4.29).

No further action is required.

Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
@Zeitsperre Zeitsperre merged commit 3c7fbc9 into main Jan 9, 2026
7 of 10 checks passed
@Zeitsperre Zeitsperre deleted the workflow-adjustments branch January 9, 2026 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Automation and Contiunous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant