Fix style checks

Author: DomAyre

src/confcom/azext_confcom/lib/opa.py

@@ -50,12 +50,3 @@ def opa_eval(data_path: Path, query: str):
         "--data", str(data_path),
         query,
     ]).stdout.strip())
-
-
-def opa_eval(data_path: Path, query: str):
-    return json.loads(opa_run([
-        "eval",
-        "--format", "json",
-        "--data", str(data_path),
-        query,
-    ]).stdout.strip())

src/confcom/azext_confcom/lib/orderless_dataclasses.py

@@ -41,8 +41,7 @@ def compare_field(name, field_info):
                         sorted(getattr(self, name), key=repr) ==
                         sorted(getattr(other, name), key=repr)
                     )
-                else:
-                    return getattr(self, name) == getattr(other, name)
+                return getattr(self, name) == getattr(other, name)
 
             return (
                 type(self) is type(other) and
@@ -58,8 +57,7 @@ def compare_field(name, field_info):
     # This adds support for using the decorator with or without parentheses.
     if cls is None:
         return wrap
-    else:
-        return wrap(cls)
+    return wrap(cls)
 
 
 def OrderlessField(**kwargs: Any):

src/confcom/azext_confcom/lib/policy.py

@@ -59,17 +59,20 @@ class ContainerMount:
 
 @dataclass
 class ContainerUser:
-    group_idnames: List[ContainerRule] = OrderlessField(default_factory=lambda: [ContainerRule(pattern="", strategy="any")])
+    group_idnames: List[ContainerRule] = \
+        OrderlessField(default_factory=lambda: [ContainerRule(pattern="", strategy="any")])
     umask: str = "0022"
-    user_idname: ContainerRule = Field(default_factory=lambda: ContainerRule(pattern="", strategy="any"))
+    user_idname: ContainerRule = \
+        Field(default_factory=lambda: ContainerRule(pattern="", strategy="any"))
 
 
 @dataclass
 class FragmentReference:
     feed: str
     issuer: str
     minimum_svn: str
-    includes: List[Literal["containers", "fragments", "namespace", "external_processes"]] = OrderlessField(default_factory=list)
+    includes: List[Literal["containers", "fragments", "namespace", "external_processes"]] = \
+        OrderlessField(default_factory=list)
     path: Optional[str] = None