MetBench run prints an error and keeps going

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#35 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
84/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
go
Bereich
cli

Rechercherichtung

Beginne mit cmd/metbench/main.go und verfolge, wie die beiden Validierungsfehler behandelt werden: n kleiner als 1 und Fehler von bench.LoadTasks(*taskDir). Gib die angeforderten Statuscodes zurück (2 für ungültiges n und 1 bei einem Fehler beim Laden der Aufgaben) und ergänze bei Bedarf gezielte Tests in einer neuen main_test.go. Fertig ist die Aufgabe, wenn beide Fehler den Lauf mit dem angegebenen Exitcode beenden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug good first issue

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.

Vorherrschende Sprache
Go
Sterne
1
Forks
2
Ø Merge
1 T. 11 Std.
Gemergte PRs (30 T.)
14

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Edwin-metcalf/MetCode

Alle Issues in Edwin-metcalf/MetCode

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.