ruff check

Author: mpharrigan

qualtran/l1/_eval.py

@@ -85,7 +85,7 @@ def eval_symbol_node(node: CObjectNode, *, safe: bool) -> sympy.Symbol:
     if len(args) != 1:
         raise TypeError(f"Symbol nodes must have one argument, not {args}")
     arg = args[0]
-    if safe and type(arg) != str:
+    if safe and type(arg) is not str:
         raise TypeError(f"Symbol nodes must take one string argument, not {arg}")
     return sympy.Symbol(args[0])
 

qualtran/l1/_parse.py

@@ -20,10 +20,10 @@
 
 import attrs
 
-logger = logging.getLogger(__name__)
-
 from .nodes import CArgNode, CObjectNode, CValueNode, L1ASTNode, LiteralNode, TupleNode
 
+logger = logging.getLogger(__name__)
+
 
 @attrs.frozen
 class Token:
@@ -134,10 +134,9 @@ def _advance_to_next_list_item(
             # Assume we already checked for an empty list and parsed an item.
             # A delimiter after an entry is always valid.
             self.advance()
-        else:
-            # A missing delimiter demands end of list
-            if not self.check(rbrack):
-                raise ValueError(f"Extraneous elements in {list_name} at {self.peek()}")
+        # A missing delimiter demands end of list
+        elif not self.check(rbrack):
+            raise ValueError(f"Extraneous elements in {list_name} at {self.peek()}")
 
         if self.check(rbrack):
             if consume_rbrack: