golang/go

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

Aperta

#25.337 aperta il 10 mag 2018

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)Go (19.008 fork)batch import
BuildersNeedsFixhelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina il codice del gerritbot in x/build/cmd/gerritbot per identificare dove vengono elaborate i messaggi di commit. Comprendi come il tool git codereview analizza il codereview.cfg e aggiungi una logica simile al bot per correggere automaticamente i messaggi di commit che contengono riferimenti a issue senza il percorso completo del repository.
Tech stack
go
Dominio
build systemci cd
Tipo issue
Bug
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGerritcodereview.cfg
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60