Skip to content

ci: use pre-commit for ci linting#110

Merged
BenjaminMummery merged 6 commits intomainfrom
test-coverage
Feb 24, 2026
Merged

ci: use pre-commit for ci linting#110
BenjaminMummery merged 6 commits intomainfrom
test-coverage

Conversation

@BenjaminMummery
Copy link
Copy Markdown
Owner

@BenjaminMummery BenjaminMummery commented Feb 24, 2026

Problem Statement

Rather than using ruff to lint the files as part of the ci pipeline, this PR replaces it with a pre-commit execution that runs all of the hooks1 with the same configuration with which they run locally.

Checklist

  • The PR title follows semantic release rules (starts with one of build, chore, ci, docs, feat,fix, perf, style, refactor, test). Note that this is CASE SENSITIVE!
  • All commits in this PR follow semantic release rules.

Footnotes

  1. Excluding upate-copyright since we only want to run that on files that have been changed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 24, 2026

✅ Result of Pytest Coverage

_______________ coverage: platform linux, python 3.14.2-final-0 ________________

Name Stmts Miss Cover
Name Stmts Miss Cover
src/_shared/comment_mapping.py 17 0 100%
src/_shared/config_parsing.py 42 0 100%
src/_shared/copyright_parsing.py 76 2 97%
src/_shared/exceptions.py 4 0 100%
src/_shared/print_diff.py 22 0 100%
src/_shared/resolvers.py 9 0 100%
src/add_copyright_hook/add_copyright.py 171 4 98%
src/add_msg_issue_hook/add_msg_issue.py 67 0 100%
src/americanise_hook/americanise.py 80 1 99%
src/no_import_testtools_in_src_hook/no_import_testtools_in_src.py 46 0 100%
src/sort_file_contents_hook/sort_file_contents.py 102 0 100%
src/update_copyright_hook/update_copyright.py 36 0 100%
TOTAL 672 7 99%
Required test coverage of

~ 867 passed in 17.15s ~

@BenjaminMummery BenjaminMummery changed the title test: improve unit test coverage ci: use pre-commit for ci linting Feb 24, 2026
@BenjaminMummery BenjaminMummery enabled auto-merge (squash) February 24, 2026 14:34
@BenjaminMummery BenjaminMummery enabled auto-merge (squash) February 24, 2026 14:42
@BenjaminMummery BenjaminMummery merged commit cf641fa into main Feb 24, 2026
23 checks passed
@BenjaminMummery BenjaminMummery deleted the test-coverage branch February 24, 2026 15:10
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