Skip to content

build(ci): Upload sentry-snapshots-runtime zip for craft (EME-1055)#1169

Merged
runningcode merged 1 commit intomainfrom
no/publish-snapshots-runtime-artifact
Apr 24, 2026
Merged

build(ci): Upload sentry-snapshots-runtime zip for craft (EME-1055)#1169
runningcode merged 1 commit intomainfrom
no/publish-snapshots-runtime-artifact

Conversation

@runningcode
Copy link
Copy Markdown
Contributor

Summary

sentry-snapshots-runtime was registered in .craft.yml under registry and the maven-publish plugin is wired up in its build.gradle.kts, but it was not published during the 6.5.0 release.

Craft's maven target publishes by downloading the per-commit zip bundle attached to the release commit by build.yml's upload-artifact step, extracting each module's pom-default.xml, and calling mvn gpg:sign-and-deploy-file. Since build.yml only assembled and uploaded zips for plugin-build and sentry-kotlin-compiler-plugin, craft never saw a sentry-snapshots-runtime.zip and silently skipped the module.

This PR adds the module to the gradle invocation and to the upload-artifact paths so it will ship on the next release.

🤖 Generated with Claude Code

Craft's maven target publishes artifacts by downloading the per-commit
zip bundle attached by build.yml and extracting each module's
pom-default.xml. The snapshots runtime was registered in .craft.yml but
never added to the build workflow, so craft never saw its zip and the
artifact was skipped during the 6.5.0 release.

Assemble and upload sentry-snapshots-runtime alongside the other
modules so it gets published on the next release.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Apr 24, 2026

@runningcode runningcode merged commit f2f447b into main Apr 24, 2026
20 checks passed
@runningcode runningcode deleted the no/publish-snapshots-runtime-artifact branch April 24, 2026 14:56
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