Skip to content

fix(ci): update release workflow triggers and runner compatibility#6

Merged
Philippoes merged 3 commits intomainfrom
fix/release-workflow-v2
Feb 6, 2026
Merged

fix(ci): update release workflow triggers and runner compatibility#6
Philippoes merged 3 commits intomainfrom
fix/release-workflow-v2

Conversation

@ajag408
Copy link
Contributor

@ajag408 ajag408 commented Feb 6, 2026

Summary

Fixes two issues preventing the release workflow from completing:

Changes

  1. Trigger: Changed from on: release: types: [published] to on: push: tags: ['v*']

    • Published releases are immutable and don't allow asset uploads
    • Tag-push trigger lets the workflow create the release and upload assets in one step
  2. macOS x64 runner: Replaced retired macos-13 with macos-latest using architecture: x64

Testing

  • Local darwin-x64 binary build verified: builds, names correctly, executes, produces valid JSON output

Copilot AI review requested due to automatic review settings February 6, 2026 00:30
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

This PR fixes two critical issues preventing the release workflow from completing successfully: updating the trigger mechanism to allow proper asset uploads and replacing the retired macOS-13 runner with a compatible alternative for building Intel Mac binaries.

Changes:

  • Modified workflow trigger from release: published to push: tags: ['v*'] to enable asset uploads during release creation
  • Replaced retired macos-13 runner with macos-latest using x64 architecture specification for Intel Mac binary builds

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

@Philippoes Philippoes merged commit 6ffeee0 into main Feb 6, 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