Skip to content

⬆️ Update runners and prek checks#582

Merged
burgholzer merged 7 commits intomainfrom
prek
Feb 2, 2026
Merged

⬆️ Update runners and prek checks#582
burgholzer merged 7 commits intomainfrom
prek

Conversation

@denialhaag
Copy link
Member

@denialhaag denialhaag commented Feb 1, 2026

Description

This PR updates the CI to use macos-15 instead of macos-14 and windows-2025 instead of windows-2022. It furthermore updates .pre-commit-config.yml to make use of prek's priority feature.

Checklist:

  • The pull request only contains commits that are focused and relevant to this change.
  • I have added appropriate tests that cover the new/changed functionality.
  • I have updated the documentation to reflect these changes.
  • I have added entries to the changelog for any noteworthy additions, changes, fixes, or removals.
  • I have added migration instructions to the upgrade guide (if needed).
  • The changes follow the project's style guidelines and introduce no new warnings.
  • The changes are fully tested and pass the CI checks.
  • I have reviewed my own code changes.

@denialhaag denialhaag self-assigned this Feb 1, 2026
@denialhaag denialhaag added dependencies Pull requests that update a dependency file continuous integration Anything related to the CI setup pre-commit labels Feb 1, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Warning

Rate limit exceeded

@burgholzer has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 57 minutes and 29 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Updated GitHub Actions workflow runner images (macos-14→macos-15, windows-2022→windows-2025, ubuntu-latest→ubuntu-slim) across multiple CI workflows. Reorganized .pre-commit-config.yaml with a priority-based hook system. Removed uv lock refresh step from noxfile and added PC170 ignore code to pyproject.toml.

Changes

Cohort / File(s) Summary
GitHub Actions Workflows
.github/workflows/ci.yml, release-drafter.yml, templating.yml
Updated runner images from legacy versions to newer ones: macos-14→macos-15, windows-2022→windows-2025, ubuntu-latest→ubuntu-slim across CI, release draft, and templating jobs.
Pre-commit Configuration
.pre-commit-config.yaml
Reorganized with priority-based hook system (Priority 0/1/2+) for staged execution. Added validation hooks (validate-pyproject, check-github-workflows, check-readthedocs, bibtex-tidy), restructured license and schema checks, and introduced local ty-check for type validation. Removed legacy hooks and streamlined overall structure.
Build & Project Configuration
noxfile.py, pyproject.toml
Removed uv lock refresh call from minimums session; added PC170 (rST files) to repo-review ignore list in pyproject.toml.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 Hop! The runners now are sleek and new,
With priorities set in order true,
From macos-15 to slim machine,
Pre-commit flows like never seen!
CI pipelines dance in staged delight,
Configuration hops—now smooth and light! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title partially relates to the changeset, accurately referencing runner updates and pre-commit checks, but uses an emoji and abbreviated terminology ('prek') that could be more explicit.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description provides a clear summary of changes (runner updates and pre-commit config reorganization) and addresses the main objectives, with most checklist items marked as completed.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch prek

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 Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@denialhaag denialhaag requested a review from burgholzer February 2, 2026 02:00
@burgholzer burgholzer enabled auto-merge (squash) February 2, 2026 12:03
@burgholzer burgholzer merged commit ae8c631 into main Feb 2, 2026
14 of 15 checks passed
@burgholzer burgholzer deleted the prek branch February 2, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

continuous integration Anything related to the CI setup dependencies Pull requests that update a dependency file pre-commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants