Skip to content

fix: update commit stages.#112

Merged
BenjaminMummery merged 1 commit intomainfrom
fix-remove-outdated-configs
Mar 2, 2026
Merged

fix: update commit stages.#112
BenjaminMummery merged 1 commit intomainfrom
fix-remove-outdated-configs

Conversation

@BenjaminMummery
Copy link
Copy Markdown
Owner

Problem Statement

pre-commit are changing the way hooks declare their stages. This PR updates our configuration to match.

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.

Base automatically changed from ci-pre-commit-skip to main March 2, 2026 13:55
@BenjaminMummery BenjaminMummery enabled auto-merge (squash) March 2, 2026 13:56
@BenjaminMummery BenjaminMummery force-pushed the fix-remove-outdated-configs branch from 7044e66 to 5f35ac0 Compare March 2, 2026 13:57
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 2, 2026

✅ Result of Pytest Coverage

_______________ coverage: platform linux, python 3.14.3-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 18.09s ~

@BenjaminMummery BenjaminMummery merged commit bf2bd76 into main Mar 2, 2026
23 checks passed
@BenjaminMummery BenjaminMummery deleted the fix-remove-outdated-configs branch March 2, 2026 14:33
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