From 881d184378f71c8e34239518ad522f9d11198962 Mon Sep 17 00:00:00 2001 From: Jason Brownstein Date: Mon, 2 Dec 2024 18:29:58 +0000 Subject: [PATCH] revert content changes --- spectacles/runner.py | 2 -- spectacles/validators/content.py | 11 ++--------- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/spectacles/runner.py b/spectacles/runner.py index c48c7969..951044ff 100644 --- a/spectacles/runner.py +++ b/spectacles/runner.py @@ -565,7 +565,6 @@ async def validate_content( name=self.project, filters=filters, include_all_explores=True, - get_full_project=get_full_project, ) explore_count = project.count_explores() print_header( @@ -573,7 +572,6 @@ async def validate_content( f"{'explore' if explore_count == 1 else 'explores'}" + (" [incremental mode] " if incremental else "") ) - await validator.validate(project) results = project.get_results(validator="content", filters=filters) diff --git a/spectacles/validators/content.py b/spectacles/validators/content.py index dfd4f0f4..72c2c2ec 100644 --- a/spectacles/validators/content.py +++ b/spectacles/validators/content.py @@ -133,10 +133,7 @@ def _get_tile_type(content: Dict[str, Any]) -> str: ) def _get_errors_from_result( - self, - project: Project, - result: Dict[str, Any], - content_type: str, + self, project: Project, result: Dict[str, Any], content_type: str ) -> List[ContentError]: content_errors: List[ContentError] = [] for error in result["errors"]: @@ -148,11 +145,7 @@ def _get_errors_from_result( else: explore = None # Skip errors that are not associated with selected explores or existing models - if not project.is_complete_project: - logger.debug( - f"Project is not complete -- showing errors for all models/explores" - ) - if explore or model or not project.is_complete_project: + if explore or model: content_id = result[content_type]["id"] folder = result[content_type].get("folder") folder_name: Optional[str] = folder.get("name") if folder else None