Skip to content

fix(ui): remove confusing None checkbox from setup options#2682

Merged
louisgv merged 1 commit intomainfrom
fix/remove-none-option
Mar 16, 2026
Merged

fix(ui): remove confusing None checkbox from setup options#2682
louisgv merged 1 commit intomainfrom
fix/remove-none-option

Conversation

@la14-1
Copy link
Member

@la14-1 la14-1 commented Mar 16, 2026

Summary

  • Remove the "None" sentinel checkbox from the setup options multiselect
  • It stayed checked alongside real selections, making it look like "None" was also selected
  • The multiselect already supports submitting with nothing selected (required: false), so the sentinel was unnecessary

Test plan

  • bunx @biomejs/biome check packages/cli/src/ — 0 errors
  • bun test — 1417 pass, 0 fail
  • Manual: run spawn → setup options should show only real options, no "None"

🤖 Generated with Claude Code

louisgv
louisgv previously approved these changes Mar 16, 2026
The "None" sentinel option stayed checked alongside real selections,
which was confusing. Remove it — the multiselect already supports
submitting with nothing selected via `required: false`.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@la14-1 la14-1 force-pushed the fix/remove-none-option branch from 1921244 to cdb6717 Compare March 16, 2026 06:40
@louisgv louisgv merged commit 09576f1 into main Mar 16, 2026
5 checks passed
@louisgv louisgv deleted the fix/remove-none-option branch March 16, 2026 06:43
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.

3 participants