Skip to content

chore: add codespell and typos pre-commit hooks, fix typos#56

Merged
jtroup merged 1 commit into
mainfrom
elmo/ideal-duck
May 7, 2026
Merged

chore: add codespell and typos pre-commit hooks, fix typos#56
jtroup merged 1 commit into
mainfrom
elmo/ideal-duck

Conversation

@jtroup
Copy link
Copy Markdown
Contributor

@jtroup jtroup commented May 6, 2026

chore: add codespell and typos pre-commit hooks, fix typos

what

Added two new spell-checking pre-commit hooks (codespell and
typos) and updated existing hooks. All pre-commit repo
revisions are now pinned to commit SHAs with a frozen-tag
comment, including bumps for pre-commit-hooks, ruff,
pyproject-fmt, mypy, actionlint, and zizmor.

Fixed typos surfaced by the new spell-checkers:

  • README.md: "wtih" -> "with"
  • .github/workflows/release.yml: "packges" -> "packages"
  • tests/test_tools_assetdb.py: "borken" -> "broken"

pyproject-fmt reordered pyproject.toml sections (pytest/coverage moved
after mypy) per the updated pyproject-fmt formatter.

Updated all dependencies with uv lock --upgrade.

why

Pinning pre-commit hooks to SHAs ensures reproducible local and CI runs
and matches the project's existing supply-chain hardening
posture. Adding codespell and typos automates catching the kind of
trivial typos that this very PR is cleaning up. Dependencies updated to
pick up CVE fixes.

testing

just lint and just test pass locally.

docs

No docs needed.


🤖 Generated with Claude Code

@jtroup jtroup requested a review from a team as a code owner May 6, 2026 23:30
@jtroup jtroup force-pushed the elmo/ideal-duck branch from 5ec4fdf to 8e3abc2 Compare May 6, 2026 23:31
Copy link
Copy Markdown
Contributor

@tvansteenburgh tvansteenburgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jtroup

@jtroup jtroup merged commit a4491e6 into main May 7, 2026
5 checks passed
@jtroup jtroup deleted the elmo/ideal-duck branch May 7, 2026 12:32
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.

2 participants