Skip to content

Commit

Permalink
Update linting (dropped Python 3.9)
Browse files Browse the repository at this point in the history
  • Loading branch information
eriknw committed Aug 27, 2024
1 parent 5a56774 commit b6f3308
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion python/nx-cugraph/lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ repos:
rev: v3.16.0
hooks:
- id: pyupgrade
args: [--py39-plus]
args: [--py310-plus]
- repo: https://github.com/psf/black
rev: 24.4.2
hooks:
Expand Down
2 changes: 2 additions & 0 deletions python/nx-cugraph/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,7 @@ external = [
]
ignore = [
# Would be nice to fix these
"B905", # `zip()` without an explicit `strict=` parameter (Note: possible since py39 was dropped; we should do this!)
"D100", # Missing docstring in public module
"D101", # Missing docstring in public class
"D102", # Missing docstring in public method
Expand Down Expand Up @@ -213,6 +214,7 @@ ignore = [
"SIM105", # Use contextlib.suppress(...) instead of try-except-pass (Note: try-except-pass is much faster)
"SIM108", # Use ternary operator ... instead of if-else-block (Note: if-else better for coverage and sometimes clearer)
"TRY003", # Avoid specifying long messages outside the exception class (Note: why?)
"UP038", # Use `X | Y` in `isinstance` call instead of `(X, Y)` (Note: tuple is faster for now)

# Ignored categories
"C90", # mccabe (Too strict, but maybe we should make things less complex)
Expand Down

0 comments on commit b6f3308

Please sign in to comment.