Remove some tests from `PLATFORM_INDEPENDENT_TESTS` by ShaharNaveh · Pull Request #7459 · RustPython/RustPython

ShaharNaveh

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: d7569199-574b-41dc-986d-f63ee9bff7be

📥 Commits

Reviewing files that changed from the base of the PR and between 0768cf8 and 404e84a.

⛔ Files ignored due to path filters (4)

Lib/test/test_class.py is excluded by !Lib/**
Lib/test/test_format.py is excluded by !Lib/**
Lib/test/test_math.py is excluded by !Lib/**
Lib/test/test_types.py is excluded by !Lib/**

📒 Files selected for processing (1)

.github/workflows/ci.yaml

💤 Files with no reviewable changes (1)

.github/workflows/ci.yaml

📝 Walkthrough

Walkthrough

Four tests (test_class, test_format, test_math, test_types) were removed from the CI workflow's Python test suite configuration. These tests will no longer execute in the corresponding CI jobs.

Changes

Cohort / File(s)	Summary
CI Test Configuration `.github/workflows/ci.yaml`	Removed four test entries (`test_class`, `test_format`, `test_math`, `test_types`) from platform-independent test list and overall test suite.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

youknowone

Poem

🐰 Four tests hop away, gone but not forgotten,
The CI workflow cleans house, no code left rotten,
Simplicity reigns where redundancy once dwelt,
A lighter test suite is how progress is felt! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and specifically describes the main change: removing tests from PLATFORM_INDEPENDENT_TESTS block in the CI workflow.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

📝 Coding Plan

Generate coding plan for human review comments

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.}