Skip to content

Update PR workflow to cover multiple Python version#184

Merged
barun-saha merged 1 commit intomainfrom
visual
Dec 5, 2025
Merged

Update PR workflow to cover multiple Python version#184
barun-saha merged 1 commit intomainfrom
visual

Conversation

@barun-saha
Copy link
Owner

@barun-saha barun-saha commented Dec 5, 2025

Close #183.

Summary by CodeRabbit

  • Chores
    • Expanded CI testing matrix to include Python 3.11 and 3.12 alongside Python 3.10, enhancing version compatibility coverage.
    • Improved test artifact organization with version-specific naming for better traceability.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 5, 2025

Walkthrough

The PR workflow is enhanced to test against multiple Python versions (3.10, 3.11, 3.12) using GitHub Actions matrix strategy, with artifact names now tied to each Python version for better organization.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/pr-workflow.yml
Expands Python version matrix from ["3.10"] to ["3.10", "3.11", "3.12"] and updates artifact naming from pytest-results to pytest-results-py${{ matrix.python-version }} to distinguish results by Python version

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three pythons now hop through our tests,
From 3.10 to twelve, we put workflows to the quest,
Each version gets its own named chest,
For artifacts proud and test results blessed!

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately describes the main change: expanding the CI workflow to test against multiple Python versions (3.10, 3.11, 3.12).
Linked Issues check ✅ Passed The PR implementation matches the linked issue #183 requirement: Python version matrix expanded to ["3.10", "3.11", "3.12"] in the CI workflow.
Out of Scope Changes check ✅ Passed All changes are scoped to the PR workflow configuration for testing multiple Python versions; no unrelated modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch visual

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 95622c7 and 7809586.

📒 Files selected for processing (1)
  • .github/workflows/pr-workflow.yml (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: test (3.10)
  • GitHub Check: test (3.12)
  • GitHub Check: test (3.11)
🔇 Additional comments (2)
.github/workflows/pr-workflow.yml (2)

12-12: ✓ Matrix expansion directly addresses PR objectives.

The Python version matrix is correctly configured to test across Python 3.10, 3.11, and 3.12 as specified. The job will run three times—once per version—with the GitHub Actions runner automatically setting the correct Python version for each execution.


36-36: ✓ Artifact naming convention prevents collisions.

Including ${{ matrix.python-version }} in the artifact name is a best practice that ensures each matrix job uploads distinct artifacts (e.g., pytest-results-py3.10, pytest-results-py3.11, pytest-results-py3.12). This prevents overwrites when jobs run in parallel and makes results easily distinguishable.


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.

@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@barun-saha barun-saha merged commit d615e21 into main Dec 5, 2025
7 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.

Run PR workflow tests for multiple Python versions

1 participant