Merge pull request #4431 from opsmill/pog-sim210

Remove unnecessary `True if ... else False` (SIM210)

backend/tests/scale/common/events.py

@@ -68,7 +68,7 @@ def request_event_handler(
         "name": name,
         "start_time": f"{start_time:.2f}",
         "response_time": f"{response_time:.2f}ms",
-        "failed": True if exception else False,
+        "failed": bool(exception),
     }
 
     if exception:

pyproject.toml

@@ -554,7 +554,6 @@ ignore = [
     "SIM117",   # Use a single `with` statement with multiple contexts instead of nested `with` statements
     "SIM118",   # Use `key in dict` instead of `key in dict.keys()`
     "SIM201",   # Use `backup_path.suffix != ".backup"` instead of `not backup_path.suffix == ".backup"`
-    "SIM210",   # Remove unnecessary `True if ... else False`
     "SIM300",   # Yoda condition detected
     "SIM401",   # Use `property["items"].get("format", None)` instead of an `if` block
     "SIM910",   #  Use `data.get("identifier")` instead of `data.get("identifier", None)`

tasks/docs.py

@@ -330,9 +330,7 @@ def _generate_infrahub_repository_configuration_documentation() -> None:
 
     for name, definition in schema["$defs"].items():
         for property, value in definition["properties"].items():
-            definitions[name]["properties"][property]["required"] = (
-                True if property in definition["required"] else False
-            )
+            definitions[name]["properties"][property]["required"] = property in definition["required"]
             if "anyOf" in value:
                 definitions[name]["properties"][property]["type"] = ", ".join(
                     [i["type"] for i in value["anyOf"] if i["type"] != "null"]