chore(ci): standardize workflows, sync maps, and meta-repo tests layout#37
Merged
buddingengineers12345 merged 7 commits intomainfrom Apr 14, 2026
Merged
Conversation
- standardize root/template workflow guardrails (permissions/concurrency/version pinning) - add template labeler workflow counterpart - lock labeler workflow in root-template sync policy/map - add committed filepaths tracker csv Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions
bot
added
documentation
- Move Copier suite to tests/integration/test_template.py and mirror scripts/ under tests/scripts/ - Add tests/_paths.py, package __init__, and pytest pythonpath for nested imports - Document layout in CLAUDE.md, rules, hooks, README; refresh freshness artifacts - Include workflow policy template sync updates on this branch Made-with: Cursor
github-actions
bot
added
dependencies
buddingengineers12345
changed the title
- Remove tests/__init__.py and tests/scripts/ — switch to flat layout
- Add tests/constants.py (renamed from _paths.py) with REPO_ROOT, TEMPLATE_ROOT, COPIER_YAML
- Add tests/conftest.py as top-level pytest config entry point
- Move all unit tests from tests/scripts/ → tests/unit/
- Add tests/integration/ and tests/e2e/ tier directories
- Update pyproject.toml pythonpath to [".", "tests"] for flat imports
- Move template test files: {{ package_name }}/ → unit/ and unit/common/
- Add template conftest.py.jinja stubs for integration/ and e2e/ tiers
- Update all imports from 'tests.constants' → 'constants' (flat)
- Fix import sort order in 3 test files (ruff I001)
- Update CLAUDE.md, tests/CLAUDE.md, .claude/rules/python/testing.md
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Delete tests/constants.py — shared constants file pattern is discouraged - Inline REPO_ROOT / TEMPLATE_ROOT / COPIER_YAML directly in each test file that needs them using Path(__file__).resolve().parent.parent.parent - Remove 'tests' from pyproject.toml pythonpath (no longer needed) - Fix PT018 compound assertion violations (new 'PT' ruleset in pyproject.toml) - Fix PT011 pytest.raises too-broad (add match= to ValueError raises) - Fix missing 'import sys' in test_sync_skip_if_exists.py - Document 'no shared constants file' rule in tests/CLAUDE.md - Update .claude/rules/python/testing.md layout tree (remove __init__.py, remove constants.py, document inline constants pattern) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- justfile / template/justfile.jinja: recipe updates - template/pyproject.toml.jinja: enhancements - scripts/bump_version.py, scripts/repo_file_freshness.py: refinements - copier.yml: variable updates - CLAUDE.md: documentation updates - .pre-commit-config.yaml: hook config tweaks Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…format The error message uses 'X.Y.Z' as a placeholder, not digits. Update match pattern from \d+\.\d+\.\d+ to X\.Y\.Z. Also update CLAUDE.md to reflect expanded ruff rule set (TCH, PGH, PT, ARG). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
2 participants
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.
Summary
tests/integration/test_template.pyfor Copier,tests/scripts/mirroringscripts/, sharedtests/_paths.py, andpythonpathin pytest config.Validation
just lintjust testjust type