Khan/genqlient

Redo error-formatting utilities to be a bit more idiomatic

Open

#118 aperta il 28 set 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Go (141 fork)github user discovery
good first issuehelp wantedinternal

Metriche repository

Star
 (1310 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

We have our own errorf (in generate/errors.go) so that we can do structured errors with positions. But our errorf is a bit un-idiomatic (which caused confusion in #117) -- it just wraps the first error argument it finds rather than using %w -- and it also doesn't actually format wrapped errors in a very good way (e.g. here). We could probably do better, either having our wrapper do an ordinary fmt.Errorf (by formatting the position at call-time), or by making it a bit more idiomatic and using its extra structure to hoist the position to the outermost error.

Guida contributor

Direzione di ricerca
Esplora i due approcci suggeriti: (1) formattare la posizione al momento della chiamata usando fmt.Errorf, oppure (2) fare in modo che errorf usi %w e porti la posizione all'errore più esterno. Guarda l'uso corrente di errorf in generate/errors.go e come gli errori sono usati altrove.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGo error handling
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75