dotnet/command-line-api

GetCompletions should check exit code of invoked application

Open

#2.137 aberto em 31 de mar. de 2023

Ver no GitHub
 (1 comment) (3 reactions) (0 assignees)C# (421 forks)auto 404
Area-Completionsbughelp wanted

Métricas do repositório

Stars
 (3.665 stars)
Métricas de merge de PR
 (Mesclagem média 1h 58m) (1 fundiu PR em 30d)

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

Guia do colaborador

Direção de pesquisa
O problema está em SuggestionStore.cs nas linhas 47-50 (vinculadas na issue). Atualmente, o código invoca o aplicativo e lê a saída padrão sem verificar o código de saída. A correção é adicionar uma verificação após a invocação: se o código de saída for diferente de zero, ignore a saída padrão. Um mantenedor confirmou que este é um bug válido. Nenhum outro arquivo precisa ser alterado.
Pilha de tecnologia
csharp
Domain
cli
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80