Skip to content

Bump strict-kwargs and use diff mode#2223

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

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

Conversation

@adamtheturtle

@adamtheturtle adamtheturtle commented May 19, 2026

Copy link
Copy Markdown
Member

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-only formatting dependency and changes the pre-commit hook to show diffs rather than modifying files; no runtime code paths are affected.

Overview
Updates developer tooling for strict-kwargs. The dev dependency is bumped to 2026.5.19.post1 and uv.lock is regenerated to match.

Pre-commit behavior change: the local strict-kwargs hook now runs strict-kwargs fix --diff, so it reports formatting changes instead of rewriting files during the hook run.

Reviewed by Cursor Bugbot for commit a24f6c8. 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:46
@adamtheturtle adamtheturtle force-pushed the chore/strict-kwargs-pre-commit branch from c4b9aaa to a24f6c8 Compare May 19, 2026 12:53
@adamtheturtle adamtheturtle merged commit e2e8af7 into main May 19, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the chore/strict-kwargs-pre-commit branch May 19, 2026 13:03
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