Skip to content

Convert CI release job to use a Matrix - #1133

Merged
nicoburns merged 1 commit into
linebender:mainfrom
nicoburns:devin/1788865643-release-matrix
Sep 9, 2026
Merged

nicoburns merged 1 commit into
linebender:mainfrom
nicoburns:devin/1788865643-release-matrix

Conversation

@nicoburns

Copy link
Copy Markdown
Collaborator

Some CI cleanup suggested by LLM review of #1132. Has been verified as working on my fork.

Changes made

  • Collapse the six per-platform release-* jobs into one release-binaries matrix job over {linux, win, macos} × {x86_64, aarch64}. Asset names are derived from the matrix (resvg-$SUFFIX.{tar.gz,zip}). This renamea resvg-win64.zip / usvg-win64.zip to resvg-win-x86_64.zip / usvg-win-x86_64.zip to match the linux-* / macos-* / win-aarch64 naming. This is a breaking change for anyone scripting downloads of the old names.
  • Drop the crates/c-api release builds (and the ilammy/msvc-dev-cmd step): their output was never collected or uploaded, so they only cost CI time.
  • Split the vendored sources archive and the Windows Explorer thumbnailer into their own jobs
  • Upload assets with softprops/action-gh-release@v3 files: (already used to create the release) instead of the third-party alexellis/upload-assets.

- Collapse the six per-platform jobs into one matrix job over
  {linux,win,macos} x {x86_64,aarch64}
- Move the vendored sources archive and the Explorer thumbnailer into
  their own jobs
- Upload assets with softprops/action-gh-release (already used to create
  the release) instead of alexellis/upload-assets
- Rename resvg-win64.zip / usvg-win64.zip to resvg-win-x86_64.zip /
  usvg-win-x86_64.zip to match the other platforms
- Drop the C API builds, whose output was never uploaded
@nicoburns
nicoburns requested a review from LaurenzV September 8, 2026 11:28

@LaurenzV LaurenzV left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you've tested it LGTM!

@nicoburns
nicoburns merged commit 2f1aa34 into linebender:main Sep 9, 2026
5 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