From dc63c8a5524b849bad5dd0fdbce392c0d58634bb Mon Sep 17 00:00:00 2001 From: Norm Brandinger Date: Sat, 13 Dec 2025 09:21:59 -0500 Subject: [PATCH] fix: resolve code scanning alerts - Replace print() with sys.stderr.write() at module level in utils.py - Remove unused Path import from vault.py Resolves GitHub code scanning alerts #5, #6, #7 --- scripts/devstack/utils.py | 4 ++-- scripts/devstack/vault.py | 1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/devstack/utils.py b/scripts/devstack/utils.py index ac43877..8373c58 100644 --- a/scripts/devstack/utils.py +++ b/scripts/devstack/utils.py @@ -28,8 +28,8 @@ from rich.console import Console from dotenv import dotenv_values except ImportError as e: - print(f"Error: Missing required dependency: {e}") - print("Install with: uv pip install -r scripts/requirements.txt") + sys.stderr.write(f"Error: Missing required dependency: {e}\n") + sys.stderr.write("Install with: uv pip install -r scripts/requirements.txt\n") sys.exit(1) # Rich console for output diff --git a/scripts/devstack/vault.py b/scripts/devstack/vault.py index 9a47125..09d34e0 100644 --- a/scripts/devstack/vault.py +++ b/scripts/devstack/vault.py @@ -20,7 +20,6 @@ from __future__ import annotations -from pathlib import Path from typing import Optional from .utils import run_command, console, VAULT_CONFIG_DIR