Skip to content

Conversation

@o-bagge
Copy link
Collaborator

@o-bagge o-bagge commented Feb 2, 2026

Refactor job if: condition to make manual triggers explicit and safe.
This prevents skipped jobs due to undefined workflow_run context while keeping the same behavior for workflow_dispatch and successful PR workflow runs.

Copy link

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

Adjusts the prerelease job gating logic so it can be run via manual dispatch in addition to successful workflow_run executions tied to PR builds.

Changes:

  • Updated the jobs.prerelease.if expression to explicitly allow workflow_dispatch.
  • Kept the workflow_run path restricted to successful runs triggered by pull_request.

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

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Build output available:
openearable_v2_firmware.elf.zip
openearable_v2_fota.zip

Copy link

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

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


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

Copy link
Collaborator

@DennisMoschina DennisMoschina left a comment

Choose a reason for hiding this comment

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

Looks good to me

@DennisMoschina DennisMoschina merged commit eaf7540 into main Feb 3, 2026
13 checks passed
@DennisMoschina DennisMoschina deleted the o-bagge-patch-1 branch February 3, 2026 12:19
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