dolthub/dolt

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

Ouverte

#11 434 ouverte le 6 août 2026

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Go (429 forks)batch import
bad error messagegood first issueversion control

Métriques du dépôt

Stars
 (15 227 étoiles)
Métriques de merge PR
 (Merge moyen 2j 21h) (58 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Inspectez le type `ExistingRefError` dans `go/libraries/doltcore/doltdb/errors.go`. Notez que `Error()` renvoie un message générique. Recherchez toutes les utilisations de `ExistingRefError` dans la base de code, en particulier dans les flux de travail de tag et d'espace de travail, et identifiez les tests qui vérifient le message d'erreur actuel. Déterminez comment inclure le champ `Ref` dans la chaîne d'erreur tout en gardant la modification minimale.
Stack technique
go
Domaine
backenddatabase
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80