From 937a1cd8476e8acf892acbb26db7b75c5bd1d226 Mon Sep 17 00:00:00 2001 From: Paul Gauthier Date: Wed, 4 Sep 2024 14:06:30 -0700 Subject: [PATCH] fix: correct string handling in ESLint file path check --- aider/linter.py | 2 +- tests/basic/test_linter.py | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/aider/linter.py b/aider/linter.py index e682c9882..b24b6d7d4 100644 --- a/aider/linter.py +++ b/aider/linter.py @@ -39,7 +39,7 @@ class Linter: if path: for name in eslint_names: eslint_file = path / name - if eslint_file.is_file() and " " not in eslint_file: + if eslint_file.is_file() and " " not in str(eslint_file): self.languages["typescript"] = f"{eslint_file} --format unix" return diff --git a/tests/basic/test_linter.py b/tests/basic/test_linter.py index ed3dbb6aa..ca8fca730 100644 --- a/tests/basic/test_linter.py +++ b/tests/basic/test_linter.py @@ -19,8 +19,7 @@ class TestLinter(unittest.TestCase): mock_is_file.return_value = True self.linter._check_eslint() self.assertIn("typescript", self.linter.languages) - self.assertTrue(self.linter.languages["typescript"].startswith('"')) - self.assertTrue(self.linter.languages["typescript"].endswith('" --format unix')) + self.assertTrue(self.linter.languages["typescript"].endswith(" --format unix")) def test_set_linter(self): self.linter.set_linter("javascript", "eslint")