Skip to content

Cachix release script - #111

Merged
RaoulSchaffranek merged 7 commits into
mainfrom
cachix-release-script
Oct 7, 2026
Merged

RaoulSchaffranek merged 7 commits into
mainfrom
cachix-release-script

Conversation

@RaoulSchaffranek

Copy link
Copy Markdown
Member

This PR replaces the legacy GitHub workflow for pushing build artefacts to the nix caches with a script that can be run locally from a developer machine.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Version metadata is inconsistent, and repository/tag validation can publish artifacts for the wrong commit or repository.

Review effort: Balanced
Findings: 1 High severity · 1 Medium severity

Open (2)
What changed in this PR

Introduces a local release workflow for publishing GitHub releases and Nix artifacts.

Changes:

  • Adds an interactive Cachix release script.
  • Adds missing Nix Python build-system overrides.
  • Advances the release version.
File Description
package/​version Advances the release tag version.
package/​release.sh Builds, caches, pins, and releases the package.
nix/​kontrol-node/​build-systems-overlay.nix Adds required Python build backends.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread package/release.sh
Comment thread package/version

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The release script performs irreversible external publishing operations that cannot be safely validated without human end-to-end verification.

Review effort: Balanced
Findings: None

Resolved since last review (2)

@RaoulSchaffranek
RaoulSchaffranek merged commit 5f862c5 into main Oct 7, 2026
6 checks passed
@RaoulSchaffranek
RaoulSchaffranek deleted the cachix-release-script branch October 7, 2026 12:26
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.

3 participants