Merge multiple isintance calls for the same object in a single call (#…

…33767)
apache · Aug 26, 2023 · 5a1889a · 5a1889a
1 parent bcb97d0
commit 5a1889a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/airflow/utils/dot_renderer.py b/airflow/utils/dot_renderer.py
@@ -120,7 +120,7 @@ def _draw_nodes(
     node: DependencyMixin, parent_graph: graphviz.Digraph, states_by_task_id: dict[str, str] | None
 ) -> None:
     """Draw the node and its children on the given parent_graph recursively."""
-    if isinstance(node, BaseOperator) or isinstance(node, MappedOperator):
+    if isinstance(node, (BaseOperator, MappedOperator)):
         _draw_task(node, parent_graph, states_by_task_id)
     else:
         if not isinstance(node, TaskGroup):

diff --git a/airflow/www/views.py b/airflow/www/views.py
@@ -4254,7 +4254,7 @@ def check_dag_edit_acl_for_actions(
             dag_ids: set[str] = set()
         elif isinstance(items, list):
             dag_ids = {item.dag_id for item in items if item is not None}
-        elif isinstance(items, TaskInstance) or isinstance(items, DagRun):
+        elif isinstance(items, (TaskInstance, DagRun)):
             dag_ids = {items.dag_id}
         else:
             raise ValueError(