Skip to content

test(ci): make classifier tests runnable from repository root - #300

Merged
seonghobae merged 2 commits into
fix/docs-only-ci-classifier-foundationfrom
codex/ci-classifier-test-import-20260909
Sep 9, 2026
Merged

test(ci): make classifier tests runnable from repository root#300
seonghobae merged 2 commits into
fix/docs-only-ci-classifier-foundationfrom
codex/ci-classifier-test-import-20260909

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

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

  • RED: the default pytest command failed during collection with ModuleNotFoundError: No module named scripts.
  • GREEN: the same default focused command passes 53 tests / 38 subtests.
  • Full repository suite: 205 tests / 581 subtests passed.

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.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: e3e1d16f-9ac7-41f2-aef9-58c1cc16161c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@seonghobae
seonghobae merged commit 3975daf into fix/docs-only-ci-classifier-foundation Sep 9, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant