Skip to content

pre-commit: add more checks#214

Merged
mpekatsoula merged 1 commit into
arm:mainfrom
fboudra:pre-commit-update
Jun 1, 2026
Merged

pre-commit: add more checks#214
mpekatsoula merged 1 commit into
arm:mainfrom
fboudra:pre-commit-update

Conversation

@fboudra
Copy link
Copy Markdown
Contributor

@fboudra fboudra commented May 31, 2026

Use ruff as a linter and formatter (ruff-check and ruff-format).
Upgrade ruff version 0.4.15 -> 0.15.5.
Drop black, replaced by ruff-formatter.
Add more pre-commit checks (check-builtin-literals, check-case-conflict,
check-docstring-first, check-toml, check-yaml, debug-statements,
detect-private-key).
Add bandit to precommit checks instead of github workflows only.
Update pyproject.toml:

  • drop black
  • add bandit
  • add ruf tool sectioni (lint.isort.force-single-line)
    pre-commit automatic fixes (src/metis/*.py))

@fboudra
Copy link
Copy Markdown
Contributor Author

fboudra commented May 31, 2026

@mpekatsoula since we already use ruff, why not using it as linter (ruff check) and formatter (ruff format)? We can then get rid of black.

@mpekatsoula
Copy link
Copy Markdown
Contributor

@fboudra Sure sounds good!

Use ruff as a linter and formatter (ruff-check and ruff-format).
Upgrade ruff version 0.4.15 -> 0.15.5.
Drop black, replaced by ruff-formatter.
Add more pre-commit checks (check-builtin-literals, check-case-conflict,
 check-docstring-first, check-toml, check-yaml, debug-statements,
 detect-private-key).
Add bandit to precommit checks instead of github workflows only.
Update pyproject.toml:
 - drop black
 - add bandit
 - add ruf tool sectioni (lint.isort.force-single-line)
pre-commit automatic fixes (src/metis/*.py))

Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>
@fboudra fboudra force-pushed the pre-commit-update branch from 21f177d to 57a599b Compare May 31, 2026 20:56
@fboudra fboudra changed the title pre-commit: run autoupdate to update black and ruff revisions pre-commit: add more checks May 31, 2026
@mpekatsoula mpekatsoula merged commit db268b6 into arm:main Jun 1, 2026
7 checks passed
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.

2 participants