Move code into src/ layout, tests into tests/ - #30
Merged
Conversation
git mv only; no logic changes. Reference updates: - pyproject.toml: pytest pythonpath=["src"], testpaths=["tests"] - Ritefile.yml: --cov=src, pylint src/*.py tests/*.py, python src/publish.py - macprefs.template.rb: libexec.install Dir["src/*"] (flatten keeps sibling imports working in the installed layout) - ci/scripts/brew-build-test.sh: PYTHONPATH=src for the version import - tests: execute_module paths now src/macprefs and src/publish.py (paths resolve relative to cwd, which is repo root under rite)
…der-using-in finding), document mise activation in README
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #28 (retargets to master when it merges).
Pure
git mvrestructure per #29 — 12 modules + themacprefsentrypoint tosrc/, 12 test files totests/— plus the coordinated reference updates: pytestpythonpath/testpaths, coverage--cov=src, pylint paths,execute_modulecall sites in tests, formulalibexec.install Dir["src/*"], andPYTHONPATH=srcin the build script.Review pass additions:
src/macprefsis now linted explicitly (its lack of a.pyextension meant it had never been linted — pylint immediately found aconsider-using-inon it, fixed here), and the README notes themise exec --fallback when mise isn't shell-activated (friction found while dogfooding the contributor bootstrap).Verified on the branch:
rite test82 passed (coverage oversrc/, 99%),rite lintclean,rite buildfull brew install + backup smoke test green.Closes #29
🤖 Generated with Claude Code