ci: run only PR-affected pytest tests - #203
Conversation
Replace the full 'uv run pytest' step with the shared minitap-ai/devops-common affected-pytest@v5 action (pytest-impacted), so PRs run only the tests their changed lines impact. Full suite still runs on push to main. Add pytest-impacted dev dep and the missing package __init__.py files so namespace dirs are discoverable by the import-graph analysis (15 new __init__.py).
|
Warning Review limit reached
More reviews will be available in 19 minutes and 20 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (17)
Comment |
1 participant
What
Make the
Feature Integration 🧩 / testcheck run only the pytest tests affected by the PR diff instead of the full suite, via the sharedaffected-pytestcomposite action (minitap-ai/devops-common/.github/actions/affected-pytest@v5).How
pytest --impacted(pytest-impacted: git-diff + AST import-graph) — only tests reachable from changed modules.mainstill runs the full suite (safety net for anything diff-selection can miss).uv.lock/pyproject.toml/ aconftest.pychange.pytest-impactedto dev deps (locked inuv.lock).__init__.pyfiles so pytest-impacto's pkgutil-based module discovery can see all source subpackages (namespace-package dirs were invisible → false negatives without this).Depends on devops-common
affected-pytest@v5(already merged + tagged).