Skip to content

Fix Pages deploy workflow by replacing failing Nix flake build path#2

Draft
gastrodon with Copilot wants to merge 4 commits into
mainfrom
copilot/fix-ci-temporary-trigger
Draft

Fix Pages deploy workflow by replacing failing Nix flake build path#2
gastrodon with Copilot wants to merge 4 commits into
mainfrom
copilot/fix-ci-temporary-trigger

Conversation

Copilot AI commented Jun 29, 2026

Copy link
Copy Markdown

Pages deploys were failing in CI after runner/environment changes exposed a brittle Nix path (buildBunPackage missing in resolved nixpkgs). This updates the workflow to use the repo’s canonical Node/Vite build path while keeping deploy semantics unchanged (push to main + workflow_dispatch).

  • Root cause removed

    • Eliminates dependence on nix build .#site for Pages artifact generation.
    • Avoids failures tied to flake evaluation/package attr drift in upstream nixpkgs.
  • Build pipeline change (workflow-only)

    • Replaces Nix bootstrap/build steps with:
      • actions/setup-node@v4 (Node 22)
      • npm ci
      • npm run build
    • Publishes ./dist directly via actions/upload-pages-artifact@v3.
  • Trigger scope preserved

    • Keeps production deploy trigger scope as push on main (plus workflow_dispatch).
    • Removes temporary branch-only trigger used during CI iteration.
- name: Setup Node.js
  uses: actions/setup-node@v4
  with:
    node-version: 22
    cache: npm

- name: Install dependencies
  run: npm ci

- name: Build site
  run: npm run build

- name: Upload artifact
  uses: actions/upload-pages-artifact@v3
  with:
    path: ./dist

Copilot AI changed the title Fix GitHub Pages CI by switching Nix installer action Stabilize Pages deploy workflow by replacing failing Nix installer action Jun 29, 2026
Copilot AI requested a review from gastrodon June 29, 2026 04:05
Copilot AI changed the title Stabilize Pages deploy workflow by replacing failing Nix installer action Fix Pages deploy workflow by replacing failing Nix flake build path Jun 29, 2026
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