Skip to content

Conversation

jamesbraza
Copy link
Collaborator

See PR title

@jamesbraza jamesbraza self-assigned this Jul 7, 2025
@Copilot Copilot AI review requested due to automatic review settings July 7, 2025 22:23
@jamesbraza jamesbraza added the enhancement New feature or request label Jul 7, 2025
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 7, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR centralizes the codespell skip configuration in pyproject.toml and updates several pre-commit hook versions and IDs.

  • Expanded codespell skip patterns in pyproject.toml
  • Bumped ruff and uv-pre-commit hook versions and adjusted the ruff hook ID
  • Removed per-hook excludes for codespell in favor of the centralized skip list

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Extended the tool.codespell.skip string to include additional paths
.pre-commit-config.yaml Upgraded ruff to v0.12.2, changed hook ID to ruff-check, bumped uv hook
Comments suppressed due to low confidence (1)

.pre-commit-config.yaml:28

  • The hook ID was changed from ruff to ruff-check, but the official hook ID in the ruff-pre-commit repo is ruff. This change may cause pre-commit to fail to locate the hook. Revert to ruff or confirm that ruff-check is a valid ID.
      - id: ruff-check

@jamesbraza jamesbraza merged commit b9a0d76 into main Jul 7, 2025
9 of 10 checks passed
@jamesbraza jamesbraza deleted the newer-tooling branch July 7, 2025 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant