Skip to content

ci(publish): finalize Sentry release after publishing#450

Merged
BYK merged 1 commit into
mainfrom
ci/finalize-sentry-release
May 21, 2026
Merged

ci(publish): finalize Sentry release after publishing#450
BYK merged 1 commit into
mainfrom
ci/finalize-sentry-release

Conversation

@BYK
Copy link
Copy Markdown
Owner

@BYK BYK commented May 21, 2026

Summary

Add Sentry release finalization to the publish workflow. Currently, sourcemaps are uploaded during CI builds but the release is never finalized — meaning Sentry doesn't track commits, deploys, or release health for published versions.

Changes

.github/workflows/publish.yml

  1. New step: "Finalize Sentry release" (after craft publish, before issue close):

    • sentry-cli releases set-commits --auto — associates commits since the previous release tag
    • sentry-cli releases finalize — marks the release as live with a dateReleased timestamp
    • sentry-cli releases deploys new -e production — creates a deploy marker for release health
  2. Added fetch-tags: true to the checkout step so set-commits --auto can find the previous release tag boundary.

  3. Uses continue-on-error: true since this is post-publish telemetry — a Sentry API outage must not block the issue from being closed or trigger false "publish failed" comments.

What this does NOT change

  • CI (ci.yml) — sourcemap upload stays where it is
  • .craft.yml — no Craft config changes needed
  • Build scripts — they continue to handle sourcemap upload with debug IDs

Add a step to the publish workflow that associates commits, finalizes the
release, and creates a production deploy marker in Sentry. Sourcemaps are
already uploaded during the CI build — this completes the release lifecycle.

Also adds fetch-tags to the checkout so set-commits --auto can find the
previous release tag boundary. The step uses continue-on-error since it's
post-publish telemetry that must not block issue closure.
@BYK BYK force-pushed the ci/finalize-sentry-release branch from 13468c6 to 27bb340 Compare May 21, 2026 13:01
@BYK BYK merged commit e489b5f into main May 21, 2026
7 checks passed
@BYK BYK deleted the ci/finalize-sentry-release branch May 21, 2026 13:08
@craft-deployer craft-deployer Bot mentioned this pull request May 21, 2026
6 tasks
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.

1 participant