Skip to content

Commit

Permalink
Merge pull request #1095 from gboeing/ruff
Browse files Browse the repository at this point in the history
linting: move ruff per-file ignored rules to the files themselves
  • Loading branch information
gboeing authored Dec 8, 2023
2 parents e9ccbb0 + 1bfe10b commit aa442f9
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 8 deletions.
1 change: 1 addition & 0 deletions osmnx/_api.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# ruff: noqa: F401
"""Expose most common parts of public API directly in package namespace."""

from .bearing import add_edge_bearings
Expand Down
8 changes: 0 additions & 8 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -112,11 +112,3 @@ max-line-length = 110 # line length + 10% since it isn't a hard upper bound

[tool.ruff.lint.pydocstyle]
convention = "numpy"

[tool.ruff.per-file-ignores]
"./osmnx/_api.py" = ["F401"] # allow API imports to go unused
"./tests/test_osmnx.py" = [
"E402", # allow imports not at the top of the file
"F841", # allow unused local variables
"PLR2004", # allow magic values in comparisons
]
1 change: 1 addition & 0 deletions tests/test_osmnx.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# ruff: noqa: E402 F841 PLR2004
"""Test suite for the package."""

# use agg backend so you don't need a display on CI
Expand Down

0 comments on commit aa442f9

Please sign in to comment.