golang/go

x/build/cmd/gerritbot: perform gofmt checks before pushing to Gerrit

Open

#24.946 aperta il 19 apr 2018

Vedi su GitHub
 (4 commenti) (1 reazione) (0 assegnatari)Go (19.008 fork)batch import
BuildersNeedsFixhelp wanted

Metriche repository

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

Descrizione

In https://golang.org/cl/99337 a file using CRLF instead of LF line endings was added. When using git-codereview to send CLs, there is a gofmt check which prevents this. But it seems there is no corresponding check when sending CLs via Github PRs.

It would be nice to run the gofmt check also for PRs submitted via Github.

/cc @bradfitz @andybons

Guida contributor

Direzione di ricerca
Esamina il codice attuale di gerritbot per capire come elabora le PR di GitHub. Aggiungi un passaggio di controllo gofmt prima di inviare a Gerrit, simile al controllo in git codereview. Assicurati che il controllo venga eseguito su tutti i file modificati e che faccia fallire la PR se gofmt segnala problemi.
Tech stack
go
Dominio
devops
Tipo issue
Funzionalità
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60