Skip to content

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.
  • 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 continuous integration Anything related to the CI setup dependencies Pull requests that update a dependency file pre-commit labels Feb 1, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Warning

Rate limit exceeded

@denialhaag has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 11 minutes and 19 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 runners and test matrix (ubuntu-latest → ubuntu-slim; macOS 14 → macOS 15; Windows 2022 → Windows 2025), reworked .pre-commit-config.yaml into priority-based hooks with many new/changed hooks, and added PC170 to [tool.repo-review] in pyproject.toml. (50 words)

Changes

Cohort / File(s) Summary
GitHub Actions workflows
\.github/workflows/ci.yml, \.github/workflows/release-drafter.yml, \.github/workflows/templating.yml
Swapped runners from ubuntu-latest to ubuntu-slim. Updated CI test matrix to include macos-15 (generic and intel) and windows-2025, removing macos-14 and windows-2022. No other workflow logic changes.
Pre-commit configuration
.pre-commit-config.yaml
Major restructure into priority-based tiers (0–3); added new public hooks (validate-pyproject, check-jsonschema group, typos, sp-repo-review, uv-lock, bibtex-tidy), a local disallow-caps hook, reorganized standard hooks and formatters (prettier, ruff-format, ruff-check, blacken-docs, mypy), and set ci.autofix_commit_msg and ci.skip: [mypy].
Project config
pyproject.toml
Added PC170 to [tool.repo-review] ignore list with comment about not using rST files.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • burgholzer

Poem

🐰 I hopped through configs, neat and prim,
Swapped ubuntu-latest to ubuntu-slim,
Brought macOS fifteen and Windows bright,
Hooks lined up in tidy light,
CI hums — the rabbit gives a trim.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title mentions updating runners and prek checks, which aligns with the main changes (CI runner updates and pre-commit configuration restructuring).
Description check ✅ Passed The description covers the key changes (CI runner updates and prek priority feature) and includes a completed checklist, though some template sections are marked as not applicable.
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
  • 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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.pre-commit-config.yaml:
- Around line 129-140: Add an explicit `priority: 2` entry to the mypy hook
block (the hook with repo "https://github.com/pre-commit/mirrors-mypy" and id
"mypy") so it is treated as a final pass consistent with other final checks;
update the existing mypy hook configuration to include the priority field at the
same indentation level as `rev` and `hooks`.

@codecov
Copy link

codecov bot commented Feb 1, 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:18
@burgholzer burgholzer enabled auto-merge (squash) February 2, 2026 12:03
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In @.pre-commit-config.yaml:
- Around line 47-55: The exclude regex for the disallow-caps hook is using
unescaped dots which match any character; update the exclude value for the hook
with id "disallow-caps" so filenames are matched precisely by escaping literal
dots (e.g., escape ".pre-commit-config.yaml" and the ".png" extension) in the
exclude pattern field to prevent accidental broad matches while keeping the
original alternation logic.
- Around line 24-29: Hooks with the same priority can run concurrently and cause
race edits; update the hook priorities so lightweight fixers run before
formatters: keep check-merge-conflict at priority 0, set end-of-file-fixer and
trailing-whitespace to priority 1, bump prettier and ruff-format to priority 2
(and any dependent checks to 3+), ensuring the hook IDs end-of-file-fixer,
trailing-whitespace, prettier, and ruff-format are reordered/assigned these
priority values so fixers run sequentially before formatters.

@burgholzer burgholzer merged commit 1b55147 into main Feb 2, 2026
19 of 20 checks passed
@burgholzer burgholzer deleted the prek branch February 2, 2026 14:36
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