golang/go

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

Offen

#25.337 geöffnet am 10.05.2018

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Go (19.008 Forks)batch import
BuildersNeedsFixhelp wanted

Repository-Metriken

Stars
 (133.883 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche den Code des gerritbot unter x/build/cmd/gerritbot, um zu identifizieren, wo Commit Nachrichten verarbeitet werden. Verstehe, wie das git codereview Tool die codereview.cfg parst, und füge dem Bot eine ähnliche Logik hinzu, um Commit Nachrichten, die Issue Referenzen ohne den vollständigen Repository Pfad enthalten, automatisch zu korrigieren.
Tech Stack
go
Domain
build systemci cd
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoGerritcodereview.cfg
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60