golang/go

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

Aberta

#25.337 aberto em 10 de mai. de 2018

Ver no GitHubGuia do colaborador
 (2 comentários) (1 reação) (0 responsável)Go (19.008 forks)batch import
BuildersNeedsFixhelp wanted

Métricas do repositório

Stars
 (133.883 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Examine o código do gerritbot em x/build/cmd/gerritbot para identificar onde as mensagens de commit são processadas. Entenda como a ferramenta git codereview analisa o codereview.cfg e adicione lógica semelhante ao bot para corrigir automaticamente mensagens de commit que contenham referências a issues sem o caminho completo do repositório.
Pilha de tecnologia
go
Domain
build systemci cd
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GoGerritcodereview.cfg
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60