dotnet/command-line-api

GetCompletions should check exit code of invoked application

Open

Aperta il 31 mar 2023

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

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

Tech stack
csharp
Dominio
cli
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
C# basicsunderstanding of exit codes
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90
Direzione di ricerca
The issue is in SuggestionStore.cs at lines 47-50 (linked in the issue). Currently, the code invokes the application and reads stdout without checking the exit code. The fix is to add a check after invocation: if exit code is non zero, ignore stdout. A maintainer has confirmed this is a valid bug. No other files need changes.