Skip to content

fix: improve smoke test conftest-hierarchy tracing in CodeRabbit instructions #5096

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rnetser merged 2 commits into from
Jun 2, 2026
Merged

fix: improve smoke test conftest-hierarchy tracing in CodeRabbit instructions #5096

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fcd20b7
fix: improve smoke test conftest-hierarchy tracing in CodeRabbit inst…
rnetser Jun 1, 2026
ea003f6
fix: resolve rule conflict in smoke test impact logic
rnetser Jun 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion .github/workflows/request-coderabbit-test-instructions.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,15 @@ jobs:
Do NOT limit impact to tests that import the modified symbol only.
7. **Smoke test impact:** Intersect the affected set from step 6 with smoke-marked tests.
Run: `rg -l '@pytest.mark.smoke' tests/`
Set True ONLY if a smoke-marked file also appears in the affected set from 6b-6e.
VERIFY the above command returned actual file paths before concluding False.
Comment thread
rnetser marked this conversation as resolved.
Set True if either condition is met:
- a smoke-marked file appears in the affected set from 6b-6e, OR
- any conftest.py in the smoke test's parent-directory hierarchy (up to repo root)
imports or calls a modified utilities/libs symbol — including autouse fixtures
that depend on modified functions. ALL tests in that directory and below are affected.
Example check: for each smoke_file, scan dirname(smoke_file)/conftest.py,
dirname(dirname(smoke_file))/conftest.py, etc. for modified symbol imports
and autouse fixtures that depend on modified symbols.
8. **Gating test impact:** Intersect the affected set from step 6 with gating-marked tests.
Run: `rg -l '@pytest.mark.gating' tests/`
Set True if a gating-marked file also appears in the affected set from 6b-6e.
Expand Down