dotnet/command-line-api

GetCompletions should check exit code of invoked application

Offen

#2.137 geöffnet am 31.03.2023

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (3 Reaktionen) (0 zugewiesene Personen)C# (421 Forks)auto 404
Area-Completionsbughelp wanted

Repository-Metriken

Stars
 (3.665 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 38T 3h) (3 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem befindet sich in SuggestionStore.cs in den Zeilen 47-50 (in der Issue verlinkt). Derzeit ruft der Code die Anwendung auf und liest die Standardausgabe, ohne den Exit Code zu prüfen. Die Lösung besteht darin, nach dem Aufruf eine Prüfung hinzuzufügen: Wenn der Exit Code ungleich Null ist, wird die Standardausgabe ignoriert. Ein Maintainer hat bestätigt, dass es sich um einen gültigen Bug handelt. Keine anderen Dateien müssen geändert werden.
Tech Stack
csharp
Domain
cli
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80