dotnet/command-line-api

GetCompletions should check exit code of invoked application

Open

#2 137 ouverte le 31 mars 2023

Voir sur GitHub
 (1 commentaire) (3 réactions) (0 assignés)C# (3 665 stars) (421 forks)auto 404
Area-Completionsbughelp wanted

Description

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

Guide contributeur

Stack technique
csharp
Domaine
cli
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
C# basicsunderstanding of exit codes
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
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.