fix: stop command missing --name flag to specify which instance to stop

Aperta Adatta ai principianti

#438 aperta il 25/5/2026 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 2/5
Tempo stimato: 1-3 ore
Idoneità per principianti: 72/100

Tipo di issue: Bug
Chiarezza: Specificata chiaramente
Stato di attività: Tranquilla
Stack tecnologico: go
Ambito: cli

Direzione di ricerca

Inizia dal punto di ingresso del comando stop e confronta i suoi flag e la risoluzione del contesto con il comando start, concentrandoti su ResolveContext(). Verifica che l’help del comando esponga --name, conferma che il nome raggiunga la risoluzione del contesto e controlla che l’arresto di un’istanza denominata funzioni senza l’output vuoto non necessario.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Why we need this:
The start command supports --name flag to name a Microcks instance
but the stop command has no --name flag. Users cannot specify which
instance to stop when multiple instances exist.

How this will help:
Adding --name flag to stop command makes it consistent with start
command and allows users to stop a specific named instance.

Problem

microcks start --name myinstance ✅ works
microcks stop --name myinstance ❌ flag does not exist

The stop command always resolves the current context instead of
allowing the user to specify which instance to stop by name.

Fix

Add --name flag to stop command
Pass name to ResolveContext() so correct instance is stopped
Remove unnecessary empty fmt.Println() call

Lingua principale: Go
Stelle: 52
Fork: 67
Merge medio: 1g 1h
PR unite (30g): 13

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.