dolthub/dolt

`ExistingRefError.Error()` omits the ref name and type

Aperta

#11.434 aperta il 6 ago 2026

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Go (429 fork)batch import
bad error messagegood first issueversion control

Metriche repository

Star
 (15.227 stelle)
Metriche merge PR
 (Merge medio 2g 21h) (58 PR mergiate in 30 g)

Descrizione

doltdb.ExistingRefError is returned when a caller-provided name collides with an existing ref. Its Error() explicitly returns a "already exists", a bare string.

https://github.com/dolthub/dolt/blob/56fc3f4f292598b644df4397d5ef1799a5883c07/go/libraries/doltcore/doltdb/errors.go#L158-L162

The type carries a Ref field that identifies the conflict. Branch creation already recovers this detail out of band, but tag and workspaces do not.

Error() should include the ref, for example: ref 'refs/heads/x' already exists. Updating 'Error()` will change the message tag and workspace callers and test assert against, so they'll have to be updated with their own wrappers.

Guida contributor

Direzione di ricerca
Ispeziona il tipo `ExistingRefError` in `go/libraries/doltcore/doltdb/errors.go`. Nota che `Error()` restituisce un messaggio generico. Cerca tutti gli usi di `ExistingRefError` nel codebase, specialmente nei flussi di lavoro di tag e workspace, e identifica i test che verificano il messaggio di errore attuale. Determina come includere il campo `Ref` nella stringa di errore mantenendo la modifica al minimo.
Tech stack
go
Dominio
backenddatabase
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80