Skip to content

Conversation

getchoo
Copy link
Member

@getchoo getchoo commented Sep 16, 2025

Motivation

Note

This is part of the discussion with the Nix and Experimental Installer team on 13-08-2025

A good way to test the new installer is to dogfood it alongside the currently existing one. This PR adds that ability to the internal install-nix-action, and implements it as a scenario to the installer test job matrix :)

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

Copy link

@adamcstephens adamcstephens left a comment

Choose a reason for hiding this comment

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

Overall this looks like a good first pass to me.

@getchoo getchoo force-pushed the getchoo/dogfood-experimental-installer branch from 43653c6 to a2fe65b Compare September 20, 2025 01:37
@getchoo getchoo force-pushed the getchoo/dogfood-experimental-installer branch from a2fe65b to 71ff2a4 Compare October 3, 2025 06:04
@getchoo getchoo marked this pull request as ready for review October 3, 2025 06:08
@getchoo getchoo requested a review from edolstra as a code owner October 3, 2025 06:08
@getchoo getchoo force-pushed the getchoo/dogfood-experimental-installer branch 4 times, most recently from 97d33af to d794c97 Compare October 3, 2025 07:17
Until these repos are potentially merged, this is good for dogfooding
alongside the experimental installer. It also uses the more official
`artifacts.nixos.org` endpoint to install stable releases now

More immediately though, we need a patch for the experimental installer
to really work in CI at all, and that hasn't landed in a tag yet. So,
this lets us use it right from `main`!
@getchoo getchoo force-pushed the getchoo/dogfood-experimental-installer branch from d794c97 to 92d7381 Compare October 3, 2025 07:44
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