Skip to content

ci: build only the site in Build and Deploy - #1302

Merged
philiptaron merged 1 commit into
NixOS:masterfrom
nixos-test-org:build-only-site
Oct 23, 2026
Merged

philiptaron merged 1 commit into
NixOS:masterfrom
nixos-test-org:build-only-site

Conversation

@philiptaron

Copy link
Copy Markdown
Collaborator

nix-build without -A builds every derivation default.nix exposes, so each master and PR build also built the development shell (Node.js, netlify-cli, devmode, ...), the update scripts and the CI tools, none of which the deploy uses. The last successful master build fetched 785 paths (1.1 GiB).

The extra downloads are not just slow: on 2026-10-23 the build of 2e48a30 failed because a transient cache.nixos.org error interrupted the download of nodejs-slim-26.12.0-npm, which only the shell needs.

./result also depended on attribute order: with several derivations, result is the first one, which was the site only because "build" sorts before the others.

nix-build without -A builds every derivation default.nix exposes, so
each master and PR build also built the development shell (Node.js,
netlify-cli, devmode, ...), the update scripts and the CI tools, none
of which the deploy uses. The last successful master build fetched 785
paths (1.1 GiB).

The extra downloads are not just slow: on 2026-10-23 the build of
2e48a30 failed because a transient cache.nixos.org error interrupted
the download of nodejs-slim-26.12.0-npm, which only the shell needs.

./result also depended on attribute order: with several derivations,
result is the first one, which was the site only because "build"
sorts before the others.
@philiptaron
philiptaron requested a review from Mic92 as a code owner October 23, 2026 09:02
@philiptaron
philiptaron merged commit 4650dbf into NixOS:master Oct 23, 2026
6 checks passed
@philiptaron
philiptaron deleted the build-only-site branch October 23, 2026 09:11
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.

1 participant