Restructure tasks/ into common/ and tests/ subpackages - #2
Merged
Conversation
- tasks/common/{main,debug,ruff,setup}.py -- main.py replaces combos.py
(same fix/test top-level aliases), debug/ruff/setup grouped alongside it.
- tasks/tests/ -- tasks/tests.py split into one file per check
(actionlint.py, pylint.py, pytest.py, rufflint.py, yamllint.py).
Both subpackages keep their tasks registered at the exact same top-level
invoke names they had before (debug.*, ruff.*, setup.*, tests.*, plus bare
fix/test) -- this is a file-layout change only, not a CLI surface change, so
every repo cloned from this template keeps working identically. Matches the
same restructuring just done in fireball_orchestrator (a downstream
consumer of this template), minus everything project-specific that lives in
template_ai_python or further downstream instead.
README.md's Project Structure section updated to match.
Generated with Claude Code
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…test - tests/setup/test_setup_properties.py, tests/style/test_markdown_style.py -- moved from a flat tests/ directory into one subfolder per concern, each with pytestmark = pytest.mark.<name> and an __init__.py. - pyproject.toml gained a markers list (setup, style) matching the folders. - tasks/tests/pytest.py gained a scope= param -- `invoke tests.pytest scope=setup` (or scope="not style") runs a marker-filtered subset instead of the whole suite. Same shape fireball_orchestrator's own tests/ already uses (one marker-named subfolder per concern, scope= for filtering). Generated with Claude Code Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Owner
Author
|
Companion PR: LevonBecker/template_ai_python#19 (adds tasks/ai/ on top of this) |
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.
Groups the flat
tasks/directory into two subpackages, keeping every task's invocation name unchanged:tasks/common/{main,debug,ruff,setup}.py—main.pyreplacescombos.py(samefix/testtop-level aliases),debug/ruff/setupgrouped alongside it.tasks/tests/—tasks/tests.pysplit into one file per check (actionlint.py,pylint.py,pytest.py,rufflint.py,yamllint.py), still registered as one flattests.*namespace.No CLI surface change —
debug.env,ruff.fix,setup.properties,tests.*,fix,testall invoke identically to before. Every repo cloned from this template keeps working without changes.Companion to the same restructuring in
template_ai_python(which addstasks/ai/on top) and already applied infireball_orchestrator(a downstream consumer).🤖 Generated with Claude Code