dotnet/tye

Exit code not properly logged in error message

Open

#1 138 ouverte le 4 août 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (532 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (5 310 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Describe the bug

If a watched replica exits due to a non-zero exit code, Tye logs an error message but does not properly include the exit code.

To Reproduce

  1. Start a Tye application in "watch" mode.
  2. Kill one of the replica processes.
  3. Note watch: Exited with error code Microsoft.Tye.ProcessResult message.

Further technical details

The bug is here; only the ExitCode property should be logged, not the whole ProcessResult.

Guide contributeur

Direction de recherche
Localisez le fichier DotNetWatcher.cs à la ligne 76 et remplacez `ProcessResult` par `ProcessResult.ExitCode` dans le message de log.
Stack technique
csharp
Domaine
backend
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.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC# basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90