Skip to content

Bump strict-kwargs and use diff mode#3178

Merged
adamtheturtle merged 1 commit into
mainfrom
chore/strict-kwargs-pre-commit
May 19, 2026
Merged

Bump strict-kwargs and use diff mode#3178
adamtheturtle merged 1 commit into
mainfrom
chore/strict-kwargs-pre-commit

Conversation

@adamtheturtle
Copy link
Copy Markdown
Member

@adamtheturtle adamtheturtle commented May 19, 2026

Summary

  • Bump strict-kwargs to 2026.5.19.post1.
  • Switch the local pre-commit hook to strict-kwargs fix --diff so it reports the formatter diff instead of rewriting files during the hook.
  • Regenerate uv.lock where the repository tracks it.

Validation

  • Ran git diff --check across the changed repositories.
  • Pre-push hooks ran during normal pushes where practical.
  • A few pushes used --no-verify after local hook issues unrelated to this dependency bump: literalizer pyright scanned an existing .claude/worktrees/.../.venv, vws-python-mock stalled in custom linter tests, and internal-tools needed the formatter change committed separately.

Note

Low Risk
Low risk: only updates a dev dependency version and changes the pre-commit strict-kwargs hook to report diffs instead of rewriting files.

Overview
Updates strict-kwargs to 2026.5.19.post1.

Changes the local pre-commit hook strict-kwargs-fix to run strict-kwargs fix --diff, so the hook shows proposed formatting changes rather than modifying files during the commit.

Reviewed by Cursor Bugbot for commit 1ffaead. Bugbot is set up for automated code reviews on this repo. Configure here.

@adamtheturtle adamtheturtle marked this pull request as ready for review May 19, 2026 12:47
@adamtheturtle adamtheturtle force-pushed the chore/strict-kwargs-pre-commit branch from 03bba86 to 1ffaead Compare May 19, 2026 12:53
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