MetBench run prints an error and keeps going

Aperta Adatta ai principianti
#35 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go
Ambito
cli

Direzione di ricerca

Inizia da cmd/metbench/main.go e segui come gestisce i due errori di convalida: n minore di 1 ed errori provenienti da bench.LoadTasks(*taskDir). Restituisci i codici di stato richiesti (2 per un valore di n non valido e 1 in caso di errore nel caricamento delle attività), quindi aggiungi test mirati in un nuovo main_test.go se possibile. Il lavoro è completato quando entrambi gli errori interrompono l’esecuzione con il codice di uscita specificato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Go
Stelle
1
Fork
2
Merge medio
1g 11h
PR unite (30g)
14

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Edwin-metcalf/MetCode

Tutte le issue di Edwin-metcalf/MetCode

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.