Skip to content

add nix flake to project#69

Merged
wvandeun merged 6 commits intomainfrom
wvd-20260209-add-nix-flake
Feb 10, 2026
Merged

add nix flake to project#69
wvandeun merged 6 commits intomainfrom
wvd-20260209-add-nix-flake

Conversation

@wvandeun
Copy link
Contributor

@wvandeun wvandeun commented Feb 9, 2026

Add flake.nix file to project + CI infrastructure to maintain the vendorHash and successful build.

@wvandeun wvandeun self-assigned this Feb 9, 2026
@wvandeun wvandeun requested a review from a team as a code owner February 9, 2026 07:19
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 9, 2026

Deploying infrahub-ops-cli with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3cb1f47
Status: ✅  Deploy successful!
Preview URL: https://a681d592.infrahub-ops-cli.pages.dev
Branch Preview URL: https://wvd-20260209-add-nix-flake.infrahub-ops-cli.pages.dev

View logs

Copy link
Contributor

@fatih-acar fatih-acar left a comment

Choose a reason for hiding this comment

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

I'm not a big fan of "auto commit within the same repo" pattern... (reasons here: opsmill/schema-library#65 (comment))

Could you switch to a "check and fail" job instead? and let users run the update-vendor-hash manually?

@wvandeun wvandeun requested a review from fatih-acar February 10, 2026 00:15
@wvandeun wvandeun requested a review from fatih-acar February 10, 2026 14:07
steps:
- uses: actions/checkout@v6
- uses: DeterminateSystems/nix-installer-action@v21
- uses: DeterminateSystems/magic-nix-cache-action@v0.1.6

Choose a reason for hiding this comment

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

Doesn't look like this tag exists.

@wvandeun wvandeun merged commit d2f92c6 into main Feb 10, 2026
9 checks passed
@wvandeun wvandeun deleted the wvd-20260209-add-nix-flake branch February 10, 2026 20:00
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