Skip to content

Conversation

shenxianpeng
Copy link
Collaborator

@shenxianpeng shenxianpeng commented Oct 7, 2025

Summary by CodeRabbit

  • Chores
    • CI now uses stable Python releases only, improving consistency and predictability of builds.
  • Tests
    • Test runs avoid prerelease Python versions, reducing flakiness and unexpected failures.
  • Documentation
    • No user-facing changes; this update affects internal build and test infrastructure to ensure more stable development and release workflows.

@shenxianpeng shenxianpeng marked this pull request as ready for review October 7, 2025 17:22
@github-actions github-actions bot added the enhancement New feature or request label Oct 7, 2025
Copy link

coderabbitai bot commented Oct 7, 2025

Walkthrough

Removed the allow-prereleases: true setting from the actions/setup-python step in .github/workflows/test.yml. No other workflow logic or files were changed.

Changes

Cohort / File(s) Summary
CI workflow
.github/workflows/test.yml
Removed the allow-prereleases: true option from the actions/setup-python step; no other edits.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The title claims to add official support for Python 3.14, but the diff only removes the allow-prereleases flag from the setup-python action in the CI workflow and does not introduce or configure Python 3.14 itself. This discrepancy means the title does not accurately summarize the main change and could mislead reviewers. A concise title should highlight the CI configuration update rather than Python version support. Please update the PR title to reflect the actual change, for example “ci: remove allow-prereleases from actions/setup-python” so that it accurately describes the modification made.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/py314

📜 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 f62d3e2 and ca38a75.

📒 Files selected for processing (1)
  • .github/workflows/test.yml (0 hunks)
💤 Files with no reviewable changes (1)
  • .github/workflows/test.yml

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.

@shenxianpeng shenxianpeng changed the title feat: offical support python 3.14 feat: official support python 3.14 Oct 7, 2025
Copy link

sonarqubecloud bot commented Oct 7, 2025

Copy link

codecov bot commented Oct 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.39%. Comparing base (f6b16ce) to head (ca38a75).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #129   +/-   ##
=======================================
  Coverage   94.39%   94.39%           
=======================================
  Files           3        3           
  Lines         107      107           
=======================================
  Hits          101      101           
  Misses          6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shenxianpeng shenxianpeng merged commit 772f7ae into main Oct 7, 2025
16 checks passed
@shenxianpeng shenxianpeng deleted the feature/py314 branch October 7, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant