Skip to content

Pre-commit CI workflow for V1#1297

Merged
ItIsUday merged 4 commits into
v1from
ci/pre-commit-v1
May 13, 2026
Merged

Pre-commit CI workflow for V1#1297
ItIsUday merged 4 commits into
v1from
ci/pre-commit-v1

Conversation

@ItIsUday
Copy link
Copy Markdown
Contributor

@ItIsUday ItIsUday commented May 7, 2026

Summary

  • Add a pre-commit CI workflow that runs on PRs and pushes to main
  • Fix pre-commit hook compatibility: switch from deprecated mirrors-autopep8 (v1.7.0, uses removed lib2to3) to official hhatto/autopep8 (v2.3.2)
  • Use Python 3.12 explicitly in the workflow since pre-commit hooks (flake8 7.0, autopep8 2.x) require >=3.8, while .python-version targets 3.6 for test compatibility

ItIsUday added 4 commits May 7, 2026 10:16
(cherry picked from commit 218ff6e)
.python-version targets 3.6 for test compatibility, but pre-commit
hooks (flake8 7.0, autopep8 2.x) require Python >=3.8. Use an explicit
Python version for the linting workflow instead of reading .python-version.
Copy link
Copy Markdown

@NicuPascu NicuPascu left a comment

Choose a reason for hiding this comment

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

given this is a pre-commit I think there's no risk in getting it in

@ItIsUday ItIsUday merged commit 56b9790 into v1 May 13, 2026
14 checks passed
@ItIsUday ItIsUday deleted the ci/pre-commit-v1 branch May 13, 2026 06:20
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