test(ci): make classifier tests runnable from repository root - #300
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
seonghobae
left a comment
There was a problem hiding this comment.
Exact-head review on 6bc57663b3bf36dd7e4a2750e2cd5d615f450cb2: the repair addresses the recorded ModuleNotFoundError: No module named scripts at repository configuration rather than preserving a shell-only PYTHONPATH=. workaround. The delta is bounded to checked-in pytest import-path configuration plus matching AGENTS/CHANGELOG documentation, with no workflow/ruleset/secret mutation. I found no browser-domain authority crossover in this patch. This is not an approval or merge claim: native CI 34315238829 is currently queued, and the PR is stacked on #287 rather than protected main, so exact terminal checks and normal parent ordering still govern promotion.
3975daf
into
fix/docs-only-ci-classifier-foundation
Outcome
Removes the shell-only
PYTHONPATH=.requirement from the protected CI scope-classifier test suite by declaring the repository root as pytest's import path.Evidence
ModuleNotFoundError: No module named scripts.Scope
Stacked on #287 because it introduces the affected classifier. No workflow, ruleset, secret, release, or publication change. This is active-PR evidence only until exact-head checks and current review complete.