Skip to content

ci(release): Switch from action-prepare-release to Craft#216

Merged
lcian merged 19 commits intomainfrom
ci/migrate-to-craft-action
Jan 23, 2026
Merged

ci(release): Switch from action-prepare-release to Craft#216
lcian merged 19 commits intomainfrom
ci/migrate-to-craft-action

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).


New Features ✨

Attributes

  • Add sentry.category attribute by mjq in #218
  • Add sentry.normalized_db_query.hash by Ahmed-Labs in #200

Other

  • (ai) Add input tokens cache write by shellmayr in #217
  • Add page listing all attributes by Lms24 in #207

Build / dependencies / internal 🔧

Attributes

  • Update sentry.span.source with known values by Lms24 in #214
  • Add legacy attribute definitions for transaction spans by Ahmed-Labs in #212
  • Normalize deprecated db attribute names by Ahmed-Labs in #199

Other

  • (ai) Remove gen_ai_usage_total_cost attribute by vgrozdanic in #206
  • (lint) Don't lint Claude's config files by mjq in #219
  • (release) Switch from action-prepare-release to Craft by BYK in #216
  • (repo) Unstale yarn lock by Lms24 in #215

🤖 This preview updates automatically when you update the PR.

BYK added 3 commits January 9, 2026 23:09
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.
Comment on lines 20 to 23
- name: Get auth token
id: token
uses: actions/create-github-app-token@d72941d797fd3113feb6b93fd0dec494b13a2547 # v2
with:

This comment was marked as outdated.

Comment on lines 30 to 36
- name: Prepare release
uses: getsentry/craft@39ee616a6a58dc64797feecb145d66770492b66c # v2
env:
GITHUB_TOKEN: ${{ steps.token.outputs.token }}
with:
version: ${{ inputs.version }}
force: ${{ inputs.force }}

This comment was marked as outdated.

Comment on lines 42 to 43
version: ${{ inputs.version }}
force: ${{ inputs.force }}

This comment was marked as outdated.

Comment on lines 41 to 46
- name: Prepare release
uses: getsentry/craft@v2
with:
version: ${{ inputs.version }}
force: ${{ inputs.force }}
secrets: inherit

This comment was marked as outdated.

Comment on lines 42 to 46
uses: getsentry/craft@1c58bfd57bfd6a967b6f3fc92bead2c42ee698ce
with:
version: ${{ inputs.version }}
force: ${{ inputs.force }}
secrets: inherit

This comment was marked as outdated.

Comment on lines 42 to 46
uses: getsentry/craft@1c58bfd57bfd6a967b6f3fc92bead2c42ee698ce
with:
version: ${{ inputs.version }}
force: ${{ inputs.force }}
secrets: inherit

This comment was marked as outdated.

Comment on lines 7 to 8
description: Version to release (or "auto")
required: false

This comment was marked as outdated.

@lcian
Copy link
Member

lcian commented Jan 15, 2026

We need node and uv installed for our script to bump versions.
From the docs it seems that the new craft workflow can only be used as a "reusable workflow", hence not in a step.
In that case, my understanding is that we won't be able to use it due to the need for node and uv to be set up.
@BYK Could you please clarify if my understanding is correct?

@lcian lcian requested a review from Lms24 as a code owner January 23, 2026 09:46
@lcian
Copy link
Member

lcian commented Jan 23, 2026

Ok everything seems to work end to end.
I've enabled auto changelog and removed danger to enforce contributors to add a changelog entry, as that won't be needed anymore.
For versioning we'll keep using manual mode for now as we still need to decide on a versioning policy and what constitutes a breaking change for this package.

@lcian lcian merged commit 3aa3194 into main Jan 23, 2026
8 of 9 checks passed
@lcian lcian deleted the ci/migrate-to-craft-action branch January 23, 2026 09:59
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