Skip to content

Commit e446580

Browse files
committed
fix: address code review comments
- Remove unused 'is_non_interactive' import in install.py - Use stderr for installation status message in install.py
1 parent 8fa0cd8 commit e446580

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/mcpm/commands/install.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
from mcpm.profile.profile_config import ProfileConfigManager
2121
from mcpm.schemas.full_server_config import FullServerConfig
2222
from mcpm.utils.config import NODE_EXECUTABLES, ConfigManager
23-
from mcpm.utils.non_interactive import should_force_operation, is_non_interactive
23+
from mcpm.utils.non_interactive import should_force_operation
2424
from mcpm.utils.repository import RepositoryManager
2525
from mcpm.utils.rich_click_config import click
2626

@@ -153,7 +153,8 @@ def install(server_name, force=False, alias=None):
153153
config_name = alias or server_name
154154

155155
# All servers are installed to global configuration
156-
console.print("[yellow]Installing server to global configuration...[/]")
156+
console_stderr = Console(stderr=True)
157+
console_stderr.print("[yellow]Installing server to global configuration...[/]")
157158

158159
# Get server metadata from repository
159160
server_metadata = repo_manager.get_server_metadata(server_name)

0 commit comments

Comments
 (0)