diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-20 16:00:39 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-20 16:00:39 -0500 |
| commit | 67d0b6ea2e132a4ac9fd25bb977dbc4cb62b3230 (patch) | |
| tree | 164ac7d44cd293820d373274406f8278c8525e7f /docs | |
| parent | db6ca439938e4a57e6d1f8d7487346a67f7c439e (diff) | |
| download | archsetup-67d0b6ea2e132a4ac9fd25bb977dbc4cb62b3230.tar.gz archsetup-67d0b6ea2e132a4ac9fd25bb977dbc4cb62b3230.zip | |
refactor(installer): extract validate_yesno and a NVIDIA driver-floor constant
Four near-identical blocks validated that AUTOLOGIN, NO_GPU_DRIVERS, INSTALL_CLAUDE_CODE, and INSTALL_DEVICE_UDEV_RULES were empty or exactly yes/no. They collapse into a validate_yesno helper the callers guard with `|| exit 1`.
The NVIDIA driver floor of 535 was a magic literal in the comparison and three prose lines. It now lives in a NVIDIA_MIN_DRIVER constant, and the nvidia-preflight test sources that constant alongside the function it extracts.
This is the safe, purely-testable slice of the refactor rollup. The structural extractions that touch the installer's system-mutation paths -- the GPU modalias scan, the stow and retry loops, the display-server/window-manager dispatch, and the snapper/fsck/GRUB/waybar sed blocks -- are split into a follow-up so they get characterization coverage first rather than a blind rewrite.
Diffstat (limited to 'docs')
0 files changed, 0 insertions, 0 deletions
