dotnet/command-line-api

GetCompletions should check exit code of invoked application

Open

#2137 aperta il 31 mar 2023

Vedi su GitHub
 (1 commento) (3 reazioni) (0 assegnatari)C# (421 fork)auto 404
Area-Completionsbughelp wanted

Metriche repository

Star
 (3665 star)
Metriche merge PR
 (Merge medio 1h 58m) (1 PR mergiata in 30 g)

Descrizione

This code should probably check the exit code and not consider standard output to be completions when it is non-zero:

https://github.com/dotnet/command-line-api/blob/403234a4c4247151d09073bc07ccc13530ce62c6/src/System.CommandLine.Suggest/SuggestionStore.cs#L47-L50

cc @jonsequitur @adamsitnik

Guida contributor

Direzione di ricerca
Il problema si trova in SuggestionStore.cs alle righe 47-50 (collegate nell'issue). Attualmente, il codice invoca l'applicazione e legge lo stdout senza controllare il codice di uscita. La correzione consiste nell'aggiungere un controllo dopo l'invocazione: se il codice di uscita è diverso da zero, ignorare stdout. Un maintainer ha confermato che si tratta di un bug valido. Non sono necessarie modifiche ad altri file.
Tech stack
csharp
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80