MetBench run prints an error and keeps going
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 84/100
Piste de recherche
Commence par cmd/metbench/main.go et suis la manière dont il traite les deux échecs de validation : n inférieur à 1 et erreurs provenant de bench.LoadTasks(*taskDir). Retourne les codes d’état demandés (2 pour une valeur de n invalide et 1 en cas d’échec du chargement des tâches), puis ajoute des tests ciblés dans un nouveau main_test.go si cela est possible. Le travail est terminé lorsque les deux échecs interrompent l’exécution avec le code de sortie spécifié.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What
Two validation failures print to stderr but don't return, so the run continues with bad input. Need to return out.
Where
cmd/metbench/main.go
Possibility to add a main_test.go as well.
Two cases
- if *n < 1
- tasks, err := bench.LoadTasks(*taskDir)
Fix
If the user puts n as 0 return 2 for the flag error (similar to the -models) and return 1 for the load error. Worth adding a test in a new main_test.go file.
- Langage dominant
- Go
- Étoiles
- 1
- Forks
- 2
- Merge moyen
- 1 j 14 h
- PR mergées (30 j)
- 17
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Edwin-metcalf/MetCode
-
stripCodeFence unti testsOuverte
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
Edwin-metcalf/MetCode#30 ·
Les mainteneurs répondent en général sous 1 jour
-
Add a code coverage badge to CIOuvertedocumentation enhancement help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Edwin-metcalf/MetCode#19 ·
Les mainteneurs répondent en général sous 1 jour
-
Add tests for resolveSafePathOuvertetesting
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Edwin-metcalf/MetCode#9 ·
Les mainteneurs répondent en général sous 1 jour
-
Add a MakefileOuverteenhancement good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Edwin-metcalf/MetCode#8 ·
Les mainteneurs répondent en général sous 1 jour
-
Add a --host flagOuverteenhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Edwin-metcalf/MetCode#7 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Edwin-metcalf/MetCode
Issues similaires
-
[v3] KeyBindingManager calls (Add/Remove) are not converted into a case-insensitive accelerator.KeyOuverteBug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
runtime: usleep sleeps nanoseconds instead of microseconds on linux/mips and linux/mips64Peut-être pris @BRobertson7 l’a pris aujourd’hui. Ouvertecompiler/runtime
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
golang/go#82913 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
bug needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
aws/aws-sdk-go-v2#3625 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
adrianbrad/queue#92 ·