Skip to content

Conversation

@rettinghaus
Copy link

@rettinghaus rettinghaus commented Oct 4, 2025

Summary by CodeRabbit

  • Chores
    • Updated CI workflows to use the latest GitHub Actions for checkout, Python setup, uv setup, and Codecov.
    • Enhances reliability, security, and compatibility with newer runners and Python versions.
    • No changes expected to build, test, or publish behavior; only underlying action versions were upgraded.
    • Streamlines coverage reporting with the latest Codecov action.
    • Maintains existing workflow structure and step sequence while modernizing dependencies.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 4, 2025

Walkthrough

Bumps GitHub Actions versions across three workflows. Updates actions/checkout to v5 and actions/setup-python to v6. In check.yml also updates astral-sh/setup-uv to v6. In unit_tests.yml updates codecov/codecov-action to v5. No changes to workflow logic, steps, or conditions.

Changes

Cohort / File(s) Summary
Workflow: Check
\.github/workflows/check.yml
Updated actions/checkout v4→v5, actions/setup-python v5→v6, astral-sh/setup-uv v5→v6. No step reordering or logic changes.
Workflow: Python Publish
\.github/workflows/python-publish.yml
Updated actions/checkout v3→v5, actions/setup-python v3→v6. No other changes.
Workflow: Unit Tests
\.github/workflows/unit_tests.yml
Updated actions/checkout v4→v5, actions/setup-python v5→v6, codecov/codecov-action v4→v5. Control flow unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • ColeDCrawford

Poem

A hop, a bump, a version tweak,
My whiskers twitch—no leaks to seek.
Checkout v5, Python v6,
CI burrows learn new tricks.
Coverage climbs, the carrots cheer—
Green meadows bright, our builds are clear. 🥕✨

Pre-merge checks and finishing touches

✅ 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 “chore: update workflow actions” succinctly and accurately captures the main change of the pull request, which is bumping the versions of GitHub Actions across multiple workflows. It is clear, concise, and directly reflects the primary update without unnecessary detail.
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

📜 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 5ef75b4 and 09d90f3.

📒 Files selected for processing (3)
  • .github/workflows/check.yml (1 hunks)
  • .github/workflows/python-publish.yml (1 hunks)
  • .github/workflows/unit_tests.yml (2 hunks)
🔇 Additional comments (2)
.github/workflows/check.yml (1)

25-28: Confirm setup-uv cache inputs still work with v6.

enable-cache and cache-dependency-glob were valid on v5, but v6 is a major release. Please confirm these inputs remain supported so caching keeps working.

Consider checking the v6 release notes or docs to verify that the cache inputs are unchanged.

.github/workflows/unit_tests.yml (1)

57-60: Double-check Codecov v5 requirements.

The jump to codecov/codecov-action@v5 may introduce new defaults or required inputs. Please verify our token-based setup still aligns with their v5 guidance so uploads don’t silently fail.

Review the v5 release notes/doc updates to confirm no additional configuration is needed for this workflow.


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 Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.93%. Comparing base (7170433) to head (09d90f3).
⚠️ Report is 33 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #147      +/-   ##
===========================================
+ Coverage    98.72%   98.93%   +0.20%     
===========================================
  Files           38       38              
  Lines         1956     2057     +101     
===========================================
+ Hits          1931     2035     +104     
+ Misses          25       22       -3     

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

@rlskoeser rlskoeser changed the base branch from main to develop October 6, 2025 20:05
@rlskoeser rlskoeser merged commit d155d25 into dh-tech:develop Oct 6, 2025
9 checks passed
@rlskoeser
Copy link
Member

@rettinghaus thank you for the update! Are you using undate for anything? Would love to know more.

@rlskoeser
Copy link
Member

@all-contributors please add @rettinghaus for maintenance

@allcontributors
Copy link
Contributor

@rlskoeser

I've put up a pull request to add @rettinghaus! 🎉

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.

2 participants