Skip to content

CI - Fix package job#4552

Merged
jdetter merged 12 commits intomasterfrom
bfops/fix-package-job
Mar 11, 2026
Merged

CI - Fix package job#4552
jdetter merged 12 commits intomasterfrom
bfops/fix-package-job

Conversation

@bfops
Copy link
Collaborator

@bfops bfops commented Mar 4, 2026

Description of Changes

We were unable to test #4473 because #githubworkflowthings, and it turns out we broke the job.

Here I fixed the job and merge the two jobs into one.

Signing still doesn't work so I've disabled the signing part for now.

API and ABI breaking changes

None. CI only.

Expected complexity level and risk

1

Testing

I ran a test job: https://github.com/clockworklabs/SpacetimeDB/actions/runs/22691343426/job/65787327828

  • Package job succeeds for non-windows
  • Package job succeeds on windows

@bfops bfops requested a review from jdetter as a code owner March 4, 2026 18:36
@bfops bfops marked this pull request as draft March 4, 2026 18:37
@bfops bfops changed the base branch from master to bfops/package-ci March 4, 2026 18:37
@bfops bfops changed the base branch from bfops/package-ci to master March 4, 2026 21:11
@bfops bfops marked this pull request as ready for review March 4, 2026 22:04
@bfops bfops marked this pull request as draft March 4, 2026 22:04
@bfops bfops marked this pull request as ready for review March 5, 2026 18:47
Copy link
Collaborator

@jdetter jdetter left a comment

Choose a reason for hiding this comment

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

Didn't test it but this seems fine for now, do we have a ticket to enable/fix signing windows binaries?

@jdetter jdetter added this pull request to the merge queue Mar 11, 2026
Merged via the queue into master with commit 0a70eb9 Mar 11, 2026
36 checks passed
@bfops bfops deleted the bfops/fix-package-job branch March 11, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants