fix: lint

Author: claytonlin1110

test_unstructured/test_utils.py

@@ -405,7 +405,8 @@ def it_sends_telemetry_when_opt_in_is_set(self, monkeypatch):
             utils.scarf_analytics()
         mock_get.assert_called_once()
         call_url = mock_get.call_args[0][0]
-        assert "python-telemetry" in call_url and "version=" in call_url
+        assert "python-telemetry" in call_url
+        assert "version=" in call_url
 
     def it_accepts_opt_in_value_1(self, monkeypatch):
         monkeypatch.setenv("UNSTRUCTURED_TELEMETRY_ENABLED", "1")

unstructured/logger.py

@@ -17,7 +17,8 @@ def detail(self, message, *args, **kws):
 
 
 # Telemetry is off by default. To opt in, set UNSTRUCTURED_TELEMETRY_ENABLED=true.
-# To opt out when enabled elsewhere, set SCARF_NO_ANALYTICS=true or DO_NOT_TRACK=true. See the README.
+# To opt out when enabled elsewhere, set SCARF_NO_ANALYTICS=true or DO_NOT_TRACK=true.
+# See the README.
 scarf_analytics()
 
 # Add the custom log method to the logging.Logger class

unstructured/utils.py

@@ -270,8 +270,9 @@ def only(it: Iterable[Any]) -> Any:
 
 
 def scarf_analytics():
-    """Send a lightweight analytics ping. Off by default; set UNSTRUCTURED_TELEMETRY_ENABLED=true to opt in.
+    """Send a lightweight analytics ping. Off by default.
 
+    Set UNSTRUCTURED_TELEMETRY_ENABLED=true to opt in.
     Opt-out env vars (DO_NOT_TRACK, SCARF_NO_ANALYTICS) are always respected and take precedence.
     """
     try:
@@ -282,8 +283,8 @@ def scarf_analytics():
 
     python_version = ".".join(platform.python_version().split(".")[:2])
 
-    # Telemetry is off by default. Only send when user explicitly opts in via UNSTRUCTURED_TELEMETRY_ENABLED.
-    # Opt-out env vars always take precedence.
+    # Telemetry is off by default. Only send when user explicitly opts in via
+    # UNSTRUCTURED_TELEMETRY_ENABLED. Opt-out env vars always take precedence.
     opt_out = os.getenv("SCARF_NO_ANALYTICS") == "true" or os.getenv("DO_NOT_TRACK") == "true"
     opt_in = os.getenv("UNSTRUCTURED_TELEMETRY_ENABLED", "").lower() in ("true", "1")
     if opt_out or not opt_in: