Skip to content

Relying on Cachix is breaking publishing process#1076

Merged
automergerpr-permission-manager[bot] merged 1 commit intomasterfrom
hotfix/kup-publish-retry
Apr 22, 2026
Merged

Relying on Cachix is breaking publishing process#1076
automergerpr-permission-manager[bot] merged 1 commit intomasterfrom
hotfix/kup-publish-retry

Conversation

@F-WRunTime
Copy link
Copy Markdown
Member

Increase timeout by 1 minute, and enable release tagging to occur and not rely on cachix responding with pin push response.

… not rely on cachix responding with pin push response.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the release workflow so release tagging/finalization is no longer gated on Cachix-related jobs, and slightly increases the Cachix pin visibility wait to reduce flakiness in the publish verification script.

Changes:

  • Remove nix-cache and nix-binary-cache from tag-release job dependencies so tagging/finalization can proceed independently.
  • Increase Cachix pin visibility timeout from 2 minutes to 3 minutes in the pin/push verification script.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/release.yml Decouples release tagging/finalization from Nix cache jobs by adjusting needs.
.github/scripts/check-cachix-pin.sh Extends retry window for detecting Cachix pin visibility.

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

Comment thread .github/workflows/release.yml
@automergerpr-permission-manager automergerpr-permission-manager Bot merged commit 434ca3e into master Apr 22, 2026
16 checks passed
@automergerpr-permission-manager automergerpr-permission-manager Bot deleted the hotfix/kup-publish-retry branch April 22, 2026 16:44
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.

3 participants