Skip to content

Add nix flake - #11

Merged
lanice merged 2 commits into
mainfrom
lanice/nix-flake
Aug 18, 2026
Merged

Add nix flake#11
lanice merged 2 commits into
mainfrom
lanice/nix-flake

Conversation

@lanice

@lanice lanice commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Adds flake.nix, together with some README + AGENTS + CI bits. The flake allows an additional method to install/use dbosctl, e.g. via nix run github:dbos-inc/dbos-ctl.

Added a step to build the nix variants in CI.

@devhawk devhawk 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.

Nice — the flake is clean and the nix flake check --all-systems note about Nix 2.19 is a good catch. Four comments inline: one I'd want addressed before merge (missing x86_64-darwin), three minor.

Also verified as fine, for the record: env.CGO_ENABLED = 0 composes correctly with buildGoModule (current module.nix merges via args.env.CGO_ENABLED or go.CGO_ENABLED); --all-systems only evaluates the non-native checks rather than building them; self.rev or self.dirtyRev or null associates as intended and the optionalString/optional guards keep isoDate lazy; dbosctl completion {bash,zsh,fish} in postInstall works since rootCmd doesn't disable Cobra's default completion command; and go_1_25 matches go.mod with no toolchain directive for GOTOOLCHAIN=local to trip over.

Comment thread flake.nix
Comment thread internal/cli/version.go
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml
@lanice
lanice merged commit 855b1e8 into main Aug 18, 2026
3 checks passed
@lanice
lanice deleted the lanice/nix-flake branch August 18, 2026 21:13
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