fix: PR Comments

sagar-salvi-apptware · Oct 2, 2024 · 04688c6 · 04688c6
1 parent 6fb8c66
commit 04688c6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/metadata-ingestion/setup.py b/metadata-ingestion/setup.py
@@ -172,8 +172,6 @@
     # LookML files with spaces between an item and the following comma.
     # See https://github.com/joshtemple/lkml/issues/73.
     "lkml>=1.3.4",
-    # Fixes https://github.com/looker-open-source/sdk-codegen/issues/1410 , cattrs appears to have introduced a breaking change starting in version 23.2.
-    "cattrs>=1.3,<23.2",
     *sqlglot_lib,
     "GitPython>2",
     "python-liquid",

diff --git a/metadata-ingestion/src/datahub/ingestion/source/looker/looker_common.py b/metadata-ingestion/src/datahub/ingestion/source/looker/looker_common.py
@@ -1136,7 +1136,7 @@ def from_api(  # noqa: C901
             logger.warning(
                 f"Failed to extract explore {explore_name} from model {model}, Error: {e}"
             )
-            reporter.report_warning(
+            reporter.warning(
                 title=f"Failed to extract explore {explore_name} from model {model}",
                 message="Encountered exception while attempting to find dependent views for this chart",
                 context=f"Explore: {explore_name}, Mode: {model}",