Skip to content

Commit 72fcee4

Browse files
committed
Merge branch 'feature/platform-login'
2 parents b0e6ade + 68d4598 commit 72fcee4

1 file changed

Lines changed: 15 additions & 7 deletions

File tree

chipfoundry_cli/main.py

Lines changed: 15 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2775,20 +2775,28 @@ def maybe_abort_no_space(err, step_label):
27752775
)
27762776
console.print("[green]✓[/green] Precheck cloned successfully")
27772777

2778-
console.print("[cyan]Pulling precheck Docker image...[/cyan]")
2779-
subprocess.run(
2780-
['docker', 'pull', 'chipfoundry/mpw_precheck:latest'],
2781-
check=True,
2782-
capture_output=True
2783-
)
2784-
console.print("[green]✓[/green] Precheck Docker image ready")
2778+
if shutil.which('docker') is None:
2779+
had_errors = True
2780+
console.print("[red]✗[/red] Docker not found. Install Docker Desktop to pull the precheck image.")
2781+
console.print("[dim]Precheck repo cloned, but Docker image was not pulled.[/dim]")
2782+
else:
2783+
console.print("[cyan]Pulling precheck Docker image...[/cyan]")
2784+
subprocess.run(
2785+
['docker', 'pull', 'chipfoundry/mpw_precheck:latest'],
2786+
check=True,
2787+
capture_output=True
2788+
)
2789+
console.print("[green]✓[/green] Precheck Docker image ready")
27852790

27862791
except subprocess.CalledProcessError as e:
27872792
maybe_abort_no_space(e, "Precheck install")
27882793
had_errors = True
27892794
console.print(f"[red]✗[/red] Failed to install precheck: {e}")
27902795
if e.stderr:
27912796
console.print(f"[dim]{e.stderr}[/dim]")
2797+
except OSError as e:
2798+
had_errors = True
2799+
console.print(f"[red]✗[/red] Failed to install precheck: {e}")
27922800

27932801
# Summary
27942802
console.print("\n" + "="*60)

0 commit comments

Comments
 (0)