Skip to content

fix: run flakehub-publish as matrix across all systems#63

Merged
sc2ben merged 2 commits into
mainfrom
61-flakehub-matrix-publish
Jun 6, 2026
Merged

fix: run flakehub-publish as matrix across all systems#63
sc2ben merged 2 commits into
mainfrom
61-flakehub-matrix-publish

Conversation

@tsunaminoai

Copy link
Copy Markdown
Contributor

Summary

  • Expand flakehub-publish from a single ubuntu-latest job into a 4-system matrix (x86_64-linux, aarch64-linux, aarch64-darwin, x86_64-darwin)
  • Each system builds and pushes its own output paths from a native runner, so macOS outputs are built on macOS instead of failing on Linux
  • Add flakehub-cache-action to each runner so they benefit from the shared cache during evaluation
  • Fixes include-output-paths: true causing zig-x86_64-macos-0.15.2 build failures on the Linux runner

Each system now builds and pushes its own output paths from a native
runner, fixing the macOS zig-0.15.2 build failure that occurred when
the single Linux runner tried to evaluate Darwin outputs with
include-output-paths: true.
Copilot AI review requested due to automatic review settings June 6, 2026 15:12
@sc2ben sc2ben linked an issue Jun 6, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the release workflow’s flakehub-publish job to publish FlakeHub artifacts across multiple native runner platforms (Linux x86_64/ARM64 and macOS ARM64/x86_64), addressing failures when macOS outputs are handled on Linux.

Changes:

  • Convert flakehub-publish from a single runner to a 4-entry OS/system matrix.
  • Add DeterminateSystems/flakehub-cache-action@v3 to flakehub-publish to leverage shared caching.
  • Run flakehub-publish on macOS runners for Darwin outputs rather than attempting them on Linux.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/release.yml
@sc2ben sc2ben enabled auto-merge (squash) June 6, 2026 15:22
@sc2ben sc2ben merged commit 53d2156 into main Jun 6, 2026
4 checks passed
@sc2ben sc2ben deleted the 61-flakehub-matrix-publish branch June 6, 2026 15:23
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.

flakehub push

3 participants