Skip to content

ci(craft): add github target so releases cut a GH release alongside npm - #38

Merged
BYK merged 1 commit into
mainfrom
chore/add-github-craft-target
Jul 31, 2026
Merged

ci(craft): add github target so releases cut a GH release alongside npm#38
BYK merged 1 commit into
mainfrom
chore/add-github-craft-target

Conversation

@BYK

@BYK BYK commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Add a github target to .craft.yml so a successful release via the
Release workflow surfaces as a normal GitHub release with the
npm-tarball CI artifact attached. The npm target alone cuts the
npm tag via OIDC trusted publishing but no GH release — making it
hard to find older releases from the GH UI.

The npm target alone cuts the npm tag via OIDC trusted publishing
but no GitHub release. Add the github target so a successful
release surfaces as a normal GH release with the npm tarball attached.
@BYK
BYK merged commit 2446f4c into main Jul 31, 2026
3 checks passed
@BYK
BYK deleted the chore/add-github-craft-target branch July 31, 2026 17:58
@github-actions

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-31 17:58 UTC

@craft-deployer craft-deployer Bot mentioned this pull request Jul 31, 2026
2 tasks
BYK added a commit to getsentry/cli that referenced this pull request Jul 31, 2026
Pulls in the synchronous output fd release from
[BYK/binpatch#37](BYK/binpatch#37) —
`applyReaderToFile` now closes the output fd via `fs.closeSync`
before returning, eliminating the ETXTBSY race that broke the
delta-upgrade → full-download fallback on Linux.

Verified locally:
- `pnpm --filter sentry run typecheck` — clean
- `pnpm --filter sentry run test:unit` — 8754 passed | 15 skipped

Also depends on
[BYK/binpatch#38](BYK/binpatch#38) (adds the
github target to .craft.yml — internal change, no runtime impact).
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