Skip to content

Clear setup state before check#102

Merged
codeforester merged 1 commit into
masterfrom
hpr/clear-check-setup-state
May 27, 2026
Merged

Clear setup state before check#102
codeforester merged 1 commit into
masterfrom
hpr/clear-check-setup-state

Conversation

@codeforester
Copy link
Copy Markdown
Owner

Summary

  • Clear inherited setup state at the start of basectl check before parsing check-specific flags.
  • Add regression coverage for ambient DRY_RUN=true and BASE_SETUP_RECREATE_VENV=true leaking into check.
  • Remove the completed TODO item.

Validation

  • bats cli/bash/commands/basectl/tests/setup.bats
  • bats cli/bash/commands/basectl/tests/basectl.bats
  • bash -n cli/bash/commands/basectl/subcommands/check.sh
  • git diff --check

@codeforester codeforester merged commit ace8bd1 into master May 27, 2026
8 checks passed
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