Skip to content

Fix release job condition#43

Merged
OGR-67 merged 2 commits into
devfrom
Fix-release-job-condition
Mar 22, 2026
Merged

Fix release job condition#43
OGR-67 merged 2 commits into
devfrom
Fix-release-job-condition

Conversation

@OGR-67

@OGR-67 OGR-67 commented Mar 22, 2026

Copy link
Copy Markdown
Owner

Description

Closes

closes #

Copilot AI review requested due to automatic review settings March 22, 2026 10:58
@OGR-67 OGR-67 merged commit 6e2d5d9 into dev Mar 22, 2026
2 of 3 checks passed
@OGR-67 OGR-67 deleted the Fix-release-job-condition branch March 22, 2026 10:58

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.

Pull request overview

Adjusts the GitHub Actions release workflow so the “Create GitHub Release” job actually runs for release-triggered executions, while skipping manual workflow dispatch runs.

Changes:

  • Update the release job condition to run when the workflow is triggered by a release event (and not when manually dispatched).

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

needs: build
runs-on: ubuntu-latest
if: github.event_name == 'push'
if: github.event_name != 'workflow_dispatch'

Copilot AI Mar 22, 2026

Copy link

Choose a reason for hiding this comment

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

For clarity and to avoid this job running unexpectedly if additional triggers are added later, consider making this condition explicit (e.g., run only when github.event_name == 'release') rather than excluding workflow_dispatch.

Suggested change
if: github.event_name != 'workflow_dispatch'
if: github.event_name == 'release'

Copilot uses AI. Check for mistakes.
needs: build
runs-on: ubuntu-latest
if: github.event_name == 'push'
if: github.event_name != 'workflow_dispatch'

Copilot AI Mar 22, 2026

Copy link

Choose a reason for hiding this comment

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

PR description currently contains a placeholder closes #. The repo's PR-body validation workflow requires closes|fixes|resolves #<issue> with a real issue number; otherwise CI will fail. Please update the PR description accordingly.

Copilot uses AI. Check for mistakes.
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