Skip to content

chore: update pre-commit hooks and development dependencies#638

Open
ceberam wants to merge 2 commits into
mainfrom
fix/test-suite
Open

chore: update pre-commit hooks and development dependencies#638
ceberam wants to merge 2 commits into
mainfrom
fix/test-suite

Conversation

@ceberam

@ceberam ceberam commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

This PR updates the development tooling infrastructure by modernizing pre-commit hooks, upgrading key dependencies, and cleaning up unused packages. It also updates the documentation to reflect the current tooling setup.

Changes

Pre-commit Configuration (.pre-commit-config.yaml)

  • Updated ruff execution versions
  • Fixed and expanded coverage: Updated file patterns to replace tests with test (bug) and add examples directories.
  • Reordered hooks: Placed ruff-check before ruff-format for logical execution order and to follow Ruff guidelines.

Development Dependencies (pyproject.toml)

  • Upgraded major versions:
    • MyPy: 1.6 → 2.1.0
  • Removed unused dependencies (Flake8, autoflake, isort,...)

Documentation (CONTRIBUTING.md)

  • Updated code style guidelines section to reflect current tooling
  • Removed references to deprecated tools (Flake8, autoflake)
  • Added link to Ruff documentation
  • Clarified that only Ruff and MyPy are used for code quality enforcement

Code Formatting

  • Applied automated code style formatting across the codebase
  • Fixed docstring formatting (raw strings where needed)
  • Removed unnecessary blank lines
  • Ensured consistency with updated style rules

Testing

  • All pre-commit hooks pass successfully
  • Code formatting has been applied consistently across the project

Impact

  • No breaking changes to the library API
  • Development workflow remains the same for contributors
  • Improved CI/CD performance with local hook execution
  • Cleaner dependency tree with removal of unused packages

@mergify

mergify Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 Require two reviewer for test updates

Waiting for

  • #approved-reviews-by >= 2
This rule is failing.

When test data is updated, we require two reviewers

  • #approved-reviews-by >= 2

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

ceberam added 2 commits June 12, 2026 10:47
Signed-off-by: Cesar Berrospi Ramis <ceb@zurich.ibm.com>
Signed-off-by: Cesar Berrospi Ramis <ceb@zurich.ibm.com>
@github-actions

Copy link
Copy Markdown
Contributor

DCO Check Passed

Thanks @ceberam, all your commits are properly signed off. 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant