remove polluting tests by youknowone · Pull Request #6733 · RustPython/RustPython

youknowone

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This change removes two test entries from environment-polluting test lists in the CI workflow configuration. Specifically, multiprocessing fork server and spawn thread tests are removed from the Linux and macOS test pollution tracking environment variables.

Changes

Cohort / File(s)	Summary
CI Workflow Configuration `.github/workflows/ci.yaml`	Removed `test.test_multiprocessing_forkserver.test_threads` from Linux environment-polluting tests and `test.test_multiprocessing_spawn.test_threads` from macOS environment-polluting tests.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Two threads hop away,
From the pollution list they'd stay,
Linux and macOS now fly free,
The CI pipeline's spree! ✨

📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e3890f9 and 9c0a9dd.

📒 Files selected for processing (1)

.github/workflows/ci.yaml

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

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 and usage tips.}