Skip to content

Commit 6be00e4

Browse files
authored
fixed ruff; made it print in CI what needs to be changed (#298)
1 parent 2575000 commit 6be00e4

File tree

4 files changed

+37
-30
lines changed

4 files changed

+37
-30
lines changed

.github/workflows/ruff_check.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,4 @@ jobs:
2424
- name: lint check and then format check with ruff
2525
run: |
2626
ruff check
27-
ruff format --check
27+
ruff format --diff

comfy_cli/cmdline.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -539,13 +539,9 @@ def which():
539539
@tracking.track_command()
540540
def env():
541541
check_for_updates()
542-
543542
env_data = EnvChecker().fill_print_table()
544-
545543
workspace_data = workspace_manager.fill_print_table()
546-
547544
all_data = env_data + workspace_data
548-
549545
ui.display_table(
550546
data=all_data,
551547
column_names=[":laptop_computer: Environment", "Value"],

comfy_cli/config_manager.py

Lines changed: 24 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,6 @@ def load(self):
8383
if not is_running(self.background[2]):
8484
self.remove_background()
8585

86-
8786
def get_env_data(self):
8887
"""
8988
Get environment data as a list of tuples for display.
@@ -96,10 +95,12 @@ def get_env_data(self):
9695

9796
launch_extras = ""
9897
if self.config.has_option("DEFAULT", "default_workspace"):
99-
data.append((
100-
"Default ComfyUI workspace",
101-
self.config["DEFAULT"][constants.CONFIG_KEY_DEFAULT_WORKSPACE],
102-
))
98+
data.append(
99+
(
100+
"Default ComfyUI workspace",
101+
self.config["DEFAULT"][constants.CONFIG_KEY_DEFAULT_WORKSPACE],
102+
)
103+
)
103104
launch_extras = self.config["DEFAULT"].get(constants.CONFIG_KEY_DEFAULT_LAUNCH_EXTRAS, "")
104105
else:
105106
data.append(("Default ComfyUI workspace", "No default ComfyUI workspace"))
@@ -110,26 +111,32 @@ def get_env_data(self):
110111
data.append(("Default ComfyUI launch extra options", launch_extras))
111112

112113
if self.config.has_option("DEFAULT", constants.CONFIG_KEY_RECENT_WORKSPACE):
113-
data.append((
114-
"Recent ComfyUI workspace",
115-
self.config["DEFAULT"][constants.CONFIG_KEY_RECENT_WORKSPACE],
116-
))
114+
data.append(
115+
(
116+
"Recent ComfyUI workspace",
117+
self.config["DEFAULT"][constants.CONFIG_KEY_RECENT_WORKSPACE],
118+
)
119+
)
117120
else:
118121
data.append(("Recent ComfyUI workspace", "No recent run"))
119122

120123
if self.config.has_option("DEFAULT", "enable_tracking"):
121-
data.append((
122-
"Tracking Analytics",
123-
("Enabled" if self.config["DEFAULT"]["enable_tracking"] == "True" else "Disabled"),
124-
))
124+
data.append(
125+
(
126+
"Tracking Analytics",
127+
("Enabled" if self.config["DEFAULT"]["enable_tracking"] == "True" else "Disabled"),
128+
)
129+
)
125130

126131
if self.config.has_option("DEFAULT", constants.CONFIG_KEY_BACKGROUND):
127132
bg_info = self.background
128133
if bg_info:
129-
data.append((
130-
"Background ComfyUI",
131-
f"http://{bg_info[0]}:{bg_info[1]} (pid={bg_info[2]})",
132-
))
134+
data.append(
135+
(
136+
"Background ComfyUI",
137+
f"http://{bg_info[0]}:{bg_info[1]} (pid={bg_info[2]})",
138+
)
139+
)
133140
else:
134141
data.append(("Background ComfyUI", "[bold red]No[/bold red]"))
135142

comfy_cli/env_checker.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -92,20 +92,24 @@ def fill_print_table(self):
9292
data = []
9393
data.append(("Python Version", format_python_version(sys.version_info)))
9494
data.append(("Python Executable", sys.executable))
95-
data.append((
96-
"Virtualenv Path",
97-
self.virtualenv_path if self.virtualenv_path else "Not Used",
98-
))
95+
data.append(
96+
(
97+
"Virtualenv Path",
98+
self.virtualenv_path if self.virtualenv_path else "Not Used",
99+
)
100+
)
99101
data.append(("Conda Env", self.conda_env if self.conda_env else "Not Used"))
100102

101103
config_data = ConfigManager().get_env_data()
102104
data.extend(config_data)
103105

104106
if check_comfy_server_running():
105-
data.append((
106-
"Comfy Server Running",
107-
"[bold green]Yes[/bold green]\nhttp://localhost:8188",
108-
))
107+
data.append(
108+
(
109+
"Comfy Server Running",
110+
"[bold green]Yes[/bold green]\nhttp://localhost:8188",
111+
)
112+
)
109113
else:
110114
data.append(("Comfy Server Running", "[bold red]No[/bold red]"))
111115

0 commit comments

Comments
 (0)