Fix logic to run lints on builds (#10304)

This was backwards - we should return None (and consequently not skip) if any Python files change.
dagster-io · Nov 2, 2022 · 3f99335 · 3f99335
1 parent 6459b0f
commit 3f99335
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.buildkite/dagster-buildkite/dagster_buildkite/package_spec.py b/.buildkite/dagster-buildkite/dagster_buildkite/package_spec.py
@@ -316,7 +316,7 @@ def skip_reason(self) -> Optional[str]:
             )
             if in_scope_changes:
 
-                logging.info(f"Building {self.name} because of changes to f{in_scope_changes}")
+                logging.info(f"Building {self.name} because of changes to {in_scope_changes}")
                 return None
 
         return "Package unaffected by these changes"
diff --git a/.buildkite/dagster-buildkite/dagster_buildkite/utils.py b/.buildkite/dagster-buildkite/dagster_buildkite/utils.py
@@ -191,7 +191,7 @@ def skip_if_no_python_changes():
     if not is_feature_branch():
         return None
 
-    if not any(path.suffix == ".py" for path in ChangedFiles.all):
+    if any(path.suffix == ".py" for path in ChangedFiles.all):
         return None
 
     return "No python changes"