Add COM812 to exclusions list for Ruff and make pre-commit check.

Author: Sherwin-14

earthaccess/store.py

@@ -298,10 +298,7 @@ def _running_in_us_west_2(self) -> bool:
         except Exception:
             return False
 
-        if resp.status_code == _HTTP_OK and resp.content == b"us-west-2":
-            # On AWS, in region us-west-2
-            return True
-        return False
+        return resp.status_code == _HTTP_OK and resp.content == b"us-west-2"
 
     def set_requests_session(self, url: str, method: str = "get") -> None:
         """Sets up a `requests` session with bearer tokens that are used by CMR.

pyproject.toml

@@ -219,6 +219,7 @@ ignore = [
   "C416",
   "C420",
   "C901",
+  "COM812",
   "D1",
   "D205",
   "D401",

tests/integration/test_api.py

@@ -260,10 +260,15 @@ def test_download_deferred_failure(tmp_path: Path):
         # raises a single generic Exception, passing the sequence of caught exceptions
         # as arguments to the Exception constructor.
         with pytest.raises(Exception) as exc_info:
-            earthaccess.download(
-                results,
-                tmp_path,
-                pqdm_kwargs=dict(exception_behaviour="deferred", disable=True),
+            (
+                (
+                    earthaccess.download(
+                        results,
+                        tmp_path,
+                        pqdm_kwargs=dict(exception_behaviour="deferred", disable=True),
+                    ),
+                )
+            )
     with (
         patch.object(earthaccess.__store__, "_download_file", fail_to_download_file),
         # With "deferred" exceptions, pqdm catches all exceptions, then at the end

tests/unit/test_virtual.py

@@ -74,7 +74,8 @@ def test_virtualize_multi_granule_no_concat_dim_raises() -> None:
 
     with (
         patch(
-            "earthaccess.virtual.core.build_obstore_registry", return_value=MagicMock(),
+            "earthaccess.virtual.core.build_obstore_registry",
+            return_value=MagicMock(),
         ),
         pytest.raises(ValueError, match="concat_dim"),
     ):
@@ -121,7 +122,8 @@ def test_virtualize_load_true_delegates_to_kerchunk(tmp_path) -> None:
         reg_patch,
         open_patch,
         patch(
-            "earthaccess.virtual.core._load_via_kerchunk", return_value=expected_ds,
+            "earthaccess.virtual.core._load_via_kerchunk",
+            return_value=expected_ds,
         ) as mock_load,
     ):
         result = virtualize(