Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Jan 9, 2026

Summary

This PR migrates from the deprecated action-prepare-release to the new Craft GitHub Actions.

Changes

  • Migrated .github/workflows/release.yml to Craft reusable workflow

Documentation

See https://getsentry.github.io/craft/github-actions/ for more information.

This PR migrates from the deprecated action-prepare-release to the new
Craft GitHub Actions (reusable workflow or composite action).

Changes:
- Migrate .github/workflows/release.yml to Craft reusable workflow
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (crashpad) Namespace mpack to avoid ODR violation by jpnurmi in #1476

Build / dependencies / internal 🔧

  • (release) Switch from action-prepare-release to Craft by BYK in #1477
  • Fix make setup in a worktree by jpnurmi in #1478

🤖 This preview updates automatically when you update the PR.

The previous migration incorrectly removed the GitHub App token
authentication step. This commit restores it by switching to the
composite action pattern which preserves the auth flow.
env:
GITHUB_TOKEN: ${{ steps.token.outputs.token }}
with:
version: ${{ inputs.version }}

This comment was marked as outdated.

Copy link
Member

@JoshuaMoelans JoshuaMoelans left a comment

Choose a reason for hiding this comment

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

(nit) I think we can remove some of the superfluous #v4 comments :)

- name: Windows (latest)
os: windows-latest
- name: Windows arm64
os: windows-11-arm
Copy link

Choose a reason for hiding this comment

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

Windows ARM64 build missing architecture specification

Medium Severity

The new Windows arm64 matrix entry specifies os: windows-11-arm but doesn't include any flag to indicate ARM64 architecture. The build scripts in tests/cmake.py and scripts/install-zlib.ps1 have explicit handling for 32-bit builds via TEST_X86 (passing -AWin32 to CMake), but no equivalent handling exists for ARM64. Without an architecture flag like -AARM64, CMake defaults to x64 even on ARM64 Windows, resulting in x64 binaries instead of native ARM64 binaries.

Fix in Cursor Fix in Web

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