Skip to content

Commit

Permalink
meta: fix remaining linter errors
Browse files Browse the repository at this point in the history
  • Loading branch information
knuxify committed May 3, 2024
1 parent 282f465 commit 2d9b037
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 8 deletions.
2 changes: 1 addition & 1 deletion src/utils/misc.py
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ def natural_compare(a: str, b: str) -> int:
if a == b:
return 0

# adapted from https://stackoverflow.com/questions/2545532/python-analog-of-phps-natsort-function-sort-a-list-using-a-natural-order-alg
# adapted from https://stackoverflow.com/questions/2545532
def sort_key(s: str) -> list:
out = []
for x in s:
Expand Down
1 change: 0 additions & 1 deletion src/window.py
Original file line number Diff line number Diff line change
Expand Up @@ -330,7 +330,6 @@ def on_drag_unhover(self, *args):

def on_drag_drop(self, drop_target, value, *args):
files = value.get_files()
paths = []

self.open_mode = EartagFileManager.LOAD_INSERT
self.load_files_from_paths([f.get_path() for f in files])
Expand Down
4 changes: 2 additions & 2 deletions tests/backends/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,8 @@ def backend_read_empty(file, skip_cover=False):
assert not file.has_tag(prop)
except AssertionError:
raise ValueError(
f"example-notags file has {prop} property set to {file.get_property(prop)}; this either means that something is broken in the file, or in the backend."
) # noqa: E501
f"example-notags file has {prop} property set to {file.get_property(prop)}; this either means that something is broken in the file, or in the backend." # noqa: E501
)

assert file.get_property("is_modified") is False
if not skip_cover:
Expand Down
4 changes: 2 additions & 2 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ def pytest_addoption(parser):
parser.addoption(
"--networked_tests",
action="store_true",
help="Enable MusicBrainz and AcoustID test; not recommended unless you're doing development on them",
) # noqa: E501
help="Enable MusicBrainz and AcoustID test; not recommended unless you're doing development on them", # noqa: E501
)


def pytest_runtest_setup(item):
Expand Down
4 changes: 2 additions & 2 deletions tests/test_musicbrainz.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,8 +113,8 @@ def test_musicbrainz_covers():

@pytest.mark.networked_tests
def test_musicbrainz_file_set(
dummy_file,
): # noqa: F811; flake8 doesn't understand fixtures
dummy_file, # noqa: F811; flake8 doesn't understand fixtures
):
"""Tests the MusicBrainz file wrappers."""

# Test with not enough data
Expand Down

0 comments on commit 2d9b037

Please sign in to comment.