Skip to content

Commit f2ac694

Browse files
committed
CI: fix E701 lint errors in niagara.py pin connect/disconnect wrappers
Split nine one-liner "if X is not None: params[K] = X" guards across connect_niagara_pins and disconnect_niagara_pins into proper two-line blocks. Ruff's default E701 (multiple statements on one line) was failing the Python Lint job on 3.10 / 3.11 / 3.12 — the pre-existing style in the rest of niagara.py already uses the split form, so this just brings these two tools in line.
1 parent 19afd84 commit f2ac694

1 file changed

Lines changed: 18 additions & 9 deletions

File tree

unrealmcp/tools/niagara.py

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2600,12 +2600,18 @@ def connect_niagara_pins(
26002600
"from_pin": from_pin,
26012601
"to_pin": to_pin,
26022602
}
2603-
if from_node_class is not None: params["from_node_class"] = from_node_class
2604-
if from_node_index is not None: params["from_node_index"] = from_node_index
2605-
if from_node_id is not None: params["from_node_id"] = from_node_id
2606-
if to_node_class is not None: params["to_node_class"] = to_node_class
2607-
if to_node_index is not None: params["to_node_index"] = to_node_index
2608-
if to_node_id is not None: params["to_node_id"] = to_node_id
2603+
if from_node_class is not None:
2604+
params["from_node_class"] = from_node_class
2605+
if from_node_index is not None:
2606+
params["from_node_index"] = from_node_index
2607+
if from_node_id is not None:
2608+
params["from_node_id"] = from_node_id
2609+
if to_node_class is not None:
2610+
params["to_node_class"] = to_node_class
2611+
if to_node_index is not None:
2612+
params["to_node_index"] = to_node_index
2613+
if to_node_id is not None:
2614+
params["to_node_id"] = to_node_id
26092615
return _call("connect_niagara_pins", params)
26102616

26112617

@@ -2624,7 +2630,10 @@ def disconnect_niagara_pins(
26242630
"module_name": module_name,
26252631
"pin_name": pin_name,
26262632
}
2627-
if node_class is not None: params["node_class"] = node_class
2628-
if node_index is not None: params["node_index"] = node_index
2629-
if node_id is not None: params["node_id"] = node_id
2633+
if node_class is not None:
2634+
params["node_class"] = node_class
2635+
if node_index is not None:
2636+
params["node_index"] = node_index
2637+
if node_id is not None:
2638+
params["node_id"] = node_id
26302639
return _call("disconnect_niagara_pins", params)

0 commit comments

Comments
 (0)