golang/go

x/build/cmd/gerritbot: does not respect codereview.cfg for fixing commit messages

Ouverte

#25 337 ouverte le 10 mai 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (1 réaction) (0 personne assignée)Go (19 008 forks)batch import
BuildersNeedsFixhelp wanted

Métriques du dépôt

Stars
 (133 883 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

https://go-review.googlesource.com/112277, which came from PR golang/vgo#4, said "Fixes #24099." The git-codereview tool would have read the vgo repo's codereview.cfg and automatically rewritten that line to say "Fixes golang/go#24099." This helps both authors and reviewers, neither of whom have to think about this anymore. The PR-bot should do the same, because as a reviewer I certainly don't want to lapse back into being in charge of catching this trivial but common mistake. Because the bot did not fix the commit message, the eventual submit did not close the issue.

Guide contributeur

Direction de recherche
Examinez le code du gerritbot dans x/build/cmd/gerritbot pour identifier où les messages de commit sont traités. Comprenez comment l'outil git codereview analyse codereview.cfg et ajoutez une logique similaire au bot pour corriger automatiquement les messages de commit contenant des références à des issues sans le chemin complet du dépôt.
Stack technique
go
Domaine
build systemci cd
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoGerritcodereview.cfg
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60