Do not log traceback if file has skip_file comment (#416)

* fix: do not log traceback if `skip_file` comment on file * [nit] update skip_file comment
microsoft · Jul 24, 2024 · 3fe5d61 · 3fe5d61
1 parent 3eef9dc
commit 3fe5d61
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/bundled/tool/lsp_server.py b/bundled/tool/lsp_server.py
@@ -650,9 +650,12 @@ def _run_tool_on_document(
                 cwd=cwd,
                 source=source,
             )
+        except isort.exceptions.FileSkipComment:
+            log_warning(f"Skipping file with 'skip_file' comment: {document.path}")
+            return None
         except isort.exceptions.FileSkipped:
             log_warning(traceback.format_exc(chain=True))
-            # Ignore file skipped.
+            return None
         except Exception:
             log_error(traceback.format_exc(chain=True))
             raise