Skip to content

Commit bdd82f4

Browse files
Address code review feedback - final score 8.27/10
Co-authored-by: SomethingGeneric <12242178+SomethingGeneric@users.noreply.github.com>
1 parent 575aa61 commit bdd82f4

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

miniupdate/os_detector.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,6 @@ def _get_architecture(self, uname_info: Dict[str, str]) -> str:
309309
return "arm"
310310
if arch.startswith("aarch64"):
311311
return "arm64"
312-
else:
313-
return arch
312+
return arch
314313

315314
return "unknown"

miniupdate/vm_mapping.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66

77
import logging
88
import os
9+
import sys
910
from pathlib import Path
1011
from typing import Dict, Any, Optional, NamedTuple
1112

@@ -125,7 +126,7 @@ def _load_mappings(self) -> Dict[str, VMMapping]:
125126

126127
except Exception as e:
127128
logger.error("Failed to load VM mappings from %s: %s", self.mapping_path, e)
128-
raise SystemExit(1) from e
129+
sys.exit(1)
129130

130131
def get_vm_info(self, host_name: str) -> Optional[VMMapping]:
131132
"""Get VM mapping for a host."""

0 commit comments

Comments
 (0)