Skip to content

⬆️ Update runners and prek checks#620

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

⬆️ Update runners and prek checks#620
burgholzer merged 6 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 labels Feb 1, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

📝 Walkthrough

Walkthrough

Updated GitHub Actions runners to newer versions across three workflows (macos-14 to macos-15, Windows-2022 to Windows-2025, ubuntu-latest to ubuntu-slim), restructured the pre-commit configuration with priority-based hook tiers, and added a single configuration ignore rule.

Changes

Cohort / File(s) Summary
GitHub Actions Runner Updates
.github/workflows/ci.yml, .github/workflows/release-drafter.yml, .github/workflows/templating.yml
Updated runner environments to newer versions: macOS 15, Windows 2025, and ubuntu-slim across test matrices and job runners.
Pre-commit Configuration Restructuring
.pre-commit-config.yaml
Reorganized hooks into three priority tiers (0, 1, 2) with explicit sequencing: fast validations, second-pass fixers, and final checks. Removed several hooks (nb-clean, texthooks, pyngrep-hooks, prettier, jsonschema, readthedocs), added validation checks (validate-pyproject, check-github-workflows, check-readthedocs), and reintroduced uv-lock with dependency constraints.
Project Configuration
pyproject.toml
Added "PC170" to tool.repo-review ignore list with note that rST files are no longer used.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • munich-quantum-toolkit/debugger#256 — Makes identical code-level changes: updates all runner versions and restructures pre-commit configuration with priority-based hooks and pyproject PC170 addition.
  • munich-quantum-toolkit/qudits#253 — Updates the same GitHub Actions runner images and reorganizes .pre-commit-config.yaml with priority/phase structure.
  • munich-quantum-toolkit/qcec#833 — Modifies CI workflow runner matrices and performs large-scale pre-commit configuration rewrites with prioritized hook phases.

Poem

🐇 The runners hop to newer ground,
With macOS-15 and Windows crowned,
Pre-commit hooks now neatly prized,
In tiers and phases organized!
Configuration fresh and light,
The CI pipeline runs just right! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: updating CI runners to newer versions and updating pre-commit checks to use prek's priority feature.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description covers the main changes and includes a populated checklist. However, it lacks specific details about the motivation, context, and dependencies for updating runners and pre-commit configuration.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 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 @.github/workflows/release-drafter.yml:
- Line 19: The workflow's runner value uses the preview-only label "ubuntu-slim"
under the runs-on key which will fail for public repos; update the runs-on entry
to a supported GitHub-hosted runner such as "ubuntu-24.04" or "ubuntu-latest" so
the release-drafter workflow can run in this public repository.

@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:07
@burgholzer burgholzer merged commit 1989680 into main Feb 2, 2026
15 of 16 checks passed
@burgholzer burgholzer deleted the prek branch February 2, 2026 12:15
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT Error Correction Feb 2, 2026
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