Skip to content

Comments

Maintenance#1360

Merged
liudger merged 3 commits intomainfrom
maintenance
Feb 23, 2026
Merged

Maintenance#1360
liudger merged 3 commits intomainfrom
maintenance

Conversation

@liudger
Copy link
Owner

@liudger liudger commented Feb 23, 2026

This pull request migrates the repository from using the pre-commit framework to prek, a faster, Rust-based drop-in replacement. All documentation, configuration files, and workflow steps have been updated to reference and use prek instead of pre-commit. Additionally, the pytest pre-commit hook was removed from the configuration, and the relevant dependency was updated in pyproject.toml.

Migration from pre-commit to prek:

  • All references to pre-commit in documentation files (README.md, .github/copilot-instructions.md, .github/prompts/add-parameter.prompt.md, .github/prompts/code-review.prompt.md, .github/skills/bsblan-parameters/SKILL.md, .github/skills/bsblan-testing/SKILL.md) have been updated to use prek instead. This includes instructions, code snippets, and section headings. [1] [2] [3] [4] [5] [6] [7]

  • The badge/link in README.md was changed from pre-commit to prek, and all usage instructions now reference prek.

Configuration and workflow changes:

  • The GitHub Actions workflow in .github/workflows/linting.yaml was updated to run prek instead of pre-commit for all hooks and checks, and the job name was changed from pre-commit-hooks to prek-hooks. [1] [2] [3] [4] [5]

  • The development dependency in pyproject.toml was updated from pre-commit to prek.

Pre-commit configuration cleanup:

  • The pytest hook was removed from .pre-commit-config.yaml, as test execution is no longer part of the (now prek-based) hook set.

Copilot AI review requested due to automatic review settings February 23, 2026 09:53
@liudger liudger added dependencies Upgrade or downgrade of project dependencies. maintenance Generic maintenance tasks. ci Work that improves the continue integration. labels Feb 23, 2026
@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.88%. Comparing base (b985ed9) to head (3776c83).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1360   +/-   ##
=======================================
  Coverage   99.88%   99.88%           
=======================================
  Files           6        6           
  Lines         898      898           
  Branches      126      126           
=======================================
  Hits          897      897           
  Partials        1        1           

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request migrates the repository from using the pre-commit framework to prek, a faster, Rust-based drop-in replacement. The migration updates all references across documentation, configuration files, and CI/CD workflows. Additionally, the pytest hook was removed from the pre-commit configuration, as test execution is now separate from the hook-based linting/formatting checks.

Changes:

  • Replaced pre-commit dependency with prek>=0.3.3 in pyproject.toml and uv.lock
  • Updated all documentation files to reference prek instead of pre-commit
  • Modified GitHub Actions workflow to use prek for all linting hooks
  • Removed pytest hook from .pre-commit-config.yaml

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
uv.lock Removed pre-commit and its dependencies (cfgv, identify, nodeenv); added prek package with platform-specific wheels
pyproject.toml Changed dev dependency from pre-commit==4.5.1 to prek>=0.3.3
README.md Updated documentation to reference prek instead of pre-commit, including badge link and usage instructions
.pre-commit-config.yaml Removed pytest hook as test execution is no longer part of the hook set
.github/workflows/linting.yaml Updated all pre-commit commands to prek and renamed job from pre-commit-hooks to prek-hooks
.github/copilot-instructions.md Updated all references from pre-commit to prek in coding guidelines and instructions
.github/prompts/code-review.prompt.md Updated validation commands to use prek
.github/prompts/add-parameter.prompt.md Updated validation commands to use prek
.github/skills/bsblan-testing/SKILL.md Updated section heading and commands from pre-commit to prek; removed pytest from hook description
.github/skills/bsblan-parameters/SKILL.md Updated validation commands to use prek

…ek' for improved command execution consistency
@sonarqubecloud
Copy link

@liudger liudger merged commit a9769ca into main Feb 23, 2026
17 checks passed
@liudger liudger deleted the maintenance branch February 23, 2026 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Work that improves the continue integration. dependencies Upgrade or downgrade of project dependencies. maintenance Generic maintenance tasks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant