Skip to content

Bump strict-kwargs and use diff mode#3012

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

Bump strict-kwargs and use diff mode#3012
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 because it only changes developer tooling (dependency version + pre-commit behavior) and does not affect runtime/library code.

Overview
Bumps the dev dependency strict-kwargs to 2026.5.19.post1.

Updates the local pre-commit strict-kwargs hook to run strict-kwargs fix --diff, so the hook reports formatting diffs rather than modifying files during the commit.

Reviewed by Cursor Bugbot for commit afd1a6e. 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 d86ad42 to afd1a6e Compare May 19, 2026 12:53
@adamtheturtle adamtheturtle merged commit 939d20c into main May 19, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the chore/strict-kwargs-pre-commit branch May 19, 2026 13:04
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