Skip to content

Refactor setup delegate modules#1482

Merged
codeforester merged 1 commit into
mainfrom
enhancement/1467-20260705-setup-delegates
Jul 5, 2026
Merged

Refactor setup delegate modules#1482
codeforester merged 1 commit into
mainfrom
enhancement/1467-20260705-setup-delegates

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • split Brewfile delegate checks/reconcile logic into base_setup.brewfile_delegate
  • split mise delegate checks/reconcile logic into base_setup.mise_delegate
  • keep base_setup.delegates as a compatibility facade with focused re-export tests

Validation

  • PYTHONPATH=lib/python:cli/python $HOME/.base.d/base/.venv/bin/python -m pytest cli/python/base_setup/tests/test_brewfile.py::BrewfileTests::test_delegates_reexports_brewfile_helpers cli/python/base_setup/tests/test_mise.py::MiseTests::test_delegates_reexports_mise_helpers -q
  • PYTHONPATH=lib/python:cli/python $HOME/.base.d/base/.venv/bin/python -m pytest cli/python/base_setup/tests/test_brewfile.py cli/python/base_setup/tests/test_mise.py -q
  • PYTHONPATH=lib/python:cli/python $HOME/.base.d/base/.venv/bin/python -m pytest cli/python/base_setup/tests -q
  • PYTHONPATH=lib/python:cli/python $HOME/.base.d/base/.venv/bin/python -m pylint cli/python/base_setup/delegates.py cli/python/base_setup/brewfile_delegate.py cli/python/base_setup/mise_delegate.py cli/python/base_setup/tests/test_brewfile.py cli/python/base_setup/tests/test_mise.py
  • python3 -m compileall -q cli/python/base_setup/delegates.py cli/python/base_setup/brewfile_delegate.py cli/python/base_setup/mise_delegate.py cli/python/base_setup/tests/test_brewfile.py cli/python/base_setup/tests/test_mise.py
  • git diff --check
  • env BASE_BASH_LIBS_DIR=/Users/rameshhp/work/base-bash-libs/lib/bash BASE_CACHE_DIR=/private/tmp/base-1467-setup-smoke bin/basectl setup base --dry-run

Closes #1467

@codeforester codeforester requested a review from a team as a code owner July 5, 2026 23:42
@codeforester codeforester merged commit a043481 into main Jul 5, 2026
26 checks passed
@codeforester codeforester deleted the enhancement/1467-20260705-setup-delegates branch July 5, 2026 23:45
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.

Split brewfile and mise delegate checks

1 participant