Skip to content

fix(ci): promote latest Release Drafter draft on tag and add tag highlights - #43

Merged
kwent merged 1 commit into
masterfrom
devin/1791387561-fix-release-job
Oct 7, 2026
Merged

kwent merged 1 commit into
masterfrom
devin/1791387561-fix-release-job

Conversation

@kwent

@kwent kwent commented Oct 7, 2026

Copy link
Copy Markdown
Member

Summary

The release job in publish.yml failed for v0.4.0 (npm publish itself succeeded):

  • It only promoted a draft whose tag_name == GITHUB_REF_NAME, but Release Drafter had resolved the draft to v0.3.1 (no breaking/enhancement labels on the merged PRs), so nothing matched.
  • The fallback gh release create then crashed with fatal: not a git repository because the job never checked out the repo.

This ports the release logic already used by rootly-swift (worked for v2.0.0):

  • check out the repo and fetch the annotated tag
  • promote the most recent draft regardless of its tag, renaming tag_name/name to the pushed tag
  • prepend the annotated tag message under ## Highlights
  • fall back to gh release create --verify-tag --generate-notes (with highlights if present) when no draft exists

Note: this does not retroactively fix v0.4.0; tag-triggered runs use the workflow at the tagged commit, so that release must be published manually from the draft.

Risk: low — release-only CI job, no SDK code changes; actionlint clean.

Link to Devin session: https://rootly.devinenterprise.com/sessions/af696ad13f4e47a09908c76ef0b67191
Open in Devin Desktop: https://rootly.devinenterprise.com/desktop/session/af696ad13f4e47a09908c76ef0b67191?variant=devin
Requested by: @kwent

…lights

Co-Authored-By: Quentin Rousseau <quentin@rootly.com>
@kwent
kwent requested a review from a team October 7, 2026 15:42
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@kwent
kwent merged commit e5412fa into master Oct 7, 2026
3 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