cake-build/cake

Tool.Run should make stdout and stderr available on failure

Offen

#2.460 geöffnet am 24.01.2019

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C# (738 Forks)batch import
Help wantedImprovementUp-for-grabs

Repository-Metriken

Stars
 (3.742 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Tool.Run and related should make stdout and stderr available so that some analysis is possible on failure. Currently it calls ProcessExitCode which throws a simple CakeException.

Ideally, it would capture stdout and stderr and throw a CakeProcessException, or similar, that has additional properties for exit code, stdout and stderr.

Contributor Guide

Research-Richtung
Die Anfrage ist, die Tool.Run Methode in src/Cake.Core/Tooling/Tool.cs so zu ändern, dass stdout und stderr vor dem Auslösen einer Exception erfasst werden. Derzeit wird in Zeile 119 ProcessExitCode aufgerufen und in Zeile 139 eine einfache CakeException ausgelöst. Die Verbesserung besteht darin, eine neue Exception Klasse CakeProcessException zu erstellen, die Exit Code, stdout und stderr enthält. Die Implementierung sollte die Standardausgabe/Standardfehler des Prozesses erfassen. Ein vorgeschlagener Ansatz ist die Verwendung von ProcessStartInfo.RedirectStandardOutput und RedirectStandardError vor dem Starten des Prozesses, dann das Lesen der Ausgabeströme vor der Überprüfung des Exit Codes. Die neue Exception sollte anstelle von CakeException ausgelöst werden. Dies ist eine klar definierte Änderung mit konkreten Dateiverweisen.
Tech Stack
csharp
Domain
build system
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70