Skip to content

⬆️ Update prek checks#95

Merged
denialhaag merged 3 commits intomainfrom
prek
Feb 2, 2026
Merged

⬆️ Update prek checks#95
denialhaag merged 3 commits intomainfrom
prek

Conversation

@denialhaag
Copy link
Member

Description

This PR 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 requested a review from burgholzer February 2, 2026 16:46
@denialhaag denialhaag self-assigned this Feb 2, 2026
@denialhaag denialhaag added dependencies Pull requests that update a dependency file pre-commit labels Feb 2, 2026
@github-project-automation github-project-automation bot moved this to In Progress in MQT Feb 2, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Summary by CodeRabbit

  • Chores

    • Updated CI workflow runners.
    • Reorganized pre-commit configuration into prioritized execution stages (fast checks, formatters, final checks).
    • Added/adjusted checks and formatters (capitalization check, license header check, spelling tool update, formatter and linter stages).
  • Documentation

    • Restructured Developers section navigation in documentation.

Walkthrough

GitHub Actions runners in two workflows are changed from ubuntu-latest to ubuntu-slim; pre-commit configuration is reorganized into priority-grouped hooks with additions/removals and version tweaks; docs/index.md toctree entries for the Developers section are restructured into hidden HTML toctree blocks.

Changes

Cohort / File(s) Summary
GitHub Actions Workflows
.github/workflows/release-drafter.yml, .github/workflows/templating.yml
Updated runs-on from ubuntu-latest to ubuntu-slim for the respective jobs (update_release_draft, render-template) with no other step changes.
Pre-commit Configuration
.pre-commit-config.yaml
Major rework introducing priority-grouped hooks (Priority 0/1/2+), added hooks (e.g., check-jsonschema, local disallow-caps, license-tools, bibtex-tidy), adjusted versions/priorities (typos, ruff/black configs), and removed several legacy hooks.
Documentation Structure
docs/index.md
Reorganized Developers section to use hidden HTML-only toctree blocks, moving overview, stats, and support items into the new structure.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐇 I hopped through CI with a twitchy grin,
swapped runners light and trimmed the bin,
hooks in tiers, lined up in rows,
docs tucked neatly where no one goes —
a tiny rabbit cheered: "Now watch it spin!"

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title refers to updating 'prek' checks, which directly aligns with the main change of updating .pre-commit-config.yml to use prek's priority feature.
Description check ✅ Passed The description clearly states the main purpose (updating .pre-commit-config.yml for prek's priority feature) and includes a completed checklist, though test and documentation updates 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
Member

@burgholzer burgholzer left a comment

Choose a reason for hiding this comment

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

Also LGTM.
I guess adding ty etc. here does not really make too much sense.

@denialhaag denialhaag merged commit 934c35f into main Feb 2, 2026
8 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT Feb 2, 2026
@denialhaag denialhaag deleted the prek branch February 2, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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