1cb203ef

Author: Michael Lynch <git@mtlynch.io>

Committer: Michael Lynch <mtlynch@noreply.codeberg.org>

Avoid Playwright dependency in small Nix checks (#170)

The shared mkBuildStep wrapper exported PLAYWRIGHT_BROWSERS_PATH for every
check derivation. That embedded the Playwright browser store path into tiny
checks like check-trailing-newline, so Nix had to realize the full Playwright
browser closure before running a shell script that only needs Git, grep, and
coreutils.

Move the Playwright environment variables into the e2e-tests setup, which is
the only build step that needs browser binaries. This keeps the e2e behavior the
same while allowing checks such as nix build .#check-trailing-newline to avoid
downloading and building Playwright browsers and their large runtime closure.

Reviewed-on: https://codeberg.org/mtlynch/little-moments/pulls/170

Suite timing

Time to Start Worker time Duration Time to finish Idle
Config 0s 3s 3s 4s 0s
Eval 3s 40s 40s 44s 0s
Build 6s 1m26s 14s 20s 0s
Suite 0s 2m10s 43s 44s 0s

Timeline

0s10s20s30s40s