Skip to content

Commit

Permalink
Fix codespell behavior. (#1769)
Browse files Browse the repository at this point in the history
Recently, I added support for `codespell` in CCCL (NVIDIA/cccl#3168). @shwina noticed some issues in my PR that were fixed in NVIDIA/cccl#3182. This PR ports similar fixes to RMM, to make `codespell` work better when run both inside and outside of `pre-commit`.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Mike Sarahan (https://github.com/msarahan)

URL: #1769
  • Loading branch information
bdice authored Dec 17, 2024
1 parent a83c2ea commit c7c38fa
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ repos:
rev: v2.3.0
hooks:
- id: codespell
additional_dependencies: [tomli]
args: ["--toml", "pyproject.toml"]
exclude: |
(?x)^(
pyproject.toml|
Expand Down
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
[tool.codespell]
# note: pre-commit passes explicit lists of files here, which this skip file list doesn't override -
# this is only to allow you to run codespell interactively
skip = "./pyproject.toml,./.git,./.github,./cpp/build,.*egg-info.*,./.mypy_cache,./benchmarks/utilities/cxxopts.hpp"
skip = "./.git,./build,./python/_skbuild/,.*egg-info.*,./.mypy_cache,./pyproject.toml,./benchmarks/utilities/cxxopts.hpp"
# ignore short words, and typename parameters like OffsetT
ignore-regex = "\\b(.{1,4}|[A-Z]\\w*T)\\b"
ignore-words-list = "inout,thirdparty"
ignore-words-list = "thirdparty,couldn"
builtin = "clear"
quiet-level = 3

Expand Down

0 comments on commit c7c38fa

Please sign in to comment.