Como configurar uma revisão de código usando Gitlab? Eu o vejo listado como um recurso no site do Gitlab, mas não consigo encontrar instruções sobre como configurá-lo (neste caso, qualquer link para um manual do usuário do Gitlab seria muito apreciado).
Algumas das minhas pesquisas indicaram que 'Solicitações de mesclagem' são o caminho a percorrer ... mas estou achando que são limitantes. Uma solicitação de mesclagem emitida mostra todos os commits entre um branch e outro. Parece que só consigo visualizar os diffs gerados para cada commit individual. Por exemplo, digamos que tenho um arquivo que desejo revisar. É um arquivo novo, mas enviei alterações nele em mais de 10 commits em um branch de desenvolvimento. Se eu emitir uma solicitação de mesclagem para esse branch dev da integração, vejo 10 commits em que cada um mostra as mudanças incrementais feitas no arquivo ... Quero revisar tudo. É novo!
Estou latindo na árvore errada aqui? Existe uma ferramenta real de revisão de código que eu possa usar no GitLab ou as solicitações de mesclagem são o caminho a percorrer e, se estiverem, estou usando-as incorretamente? qual é a melhor maneira de configurar uma revisão de código adequada aqui?
fonte
Respostas:
Observação: desde GitLab 6.4, a visualização de comparação lado a lado está disponível: consulte " solicitação de pull 5308 ".
(Julho de 2013)
Ainda não há possibilidade de comentar em cada linha, apenas no nível do arquivo.Daniel Sokolowski menciona nos comentários que os comentários por linha agora são suportados (09/2014):
Isso ainda pode ajudar na atividade de revisão de código.
6 anos depois, para GitLab 13.1 (junho de 2020) :
Veja a documentação e o problema
fonte
Tenho feito revisões de código no Gitlab por mais de dois meses quase sem atrito. Eu configurei o rss2email para enviar notificações por e-mail toda vez que um desenvolvedor envia novos commits. Então eu uso o recurso de comentário do Gitlab para commits para fazer alguns comentários sobre o código enviado.
Infelizmente, o Gitlab não permite comentários nos arquivos em si, apenas em commits (assim como o Github, eu acho). Sempre que me encontro em uma situação em que preciso comentar algo que perdi em um commit anterior, uso a ferramenta de culpa para encontrar o commit que introduziu / alterou a seção de código a ser comentada.
Está longe de ser perfeito, mas está funcionando bem até agora.
fonte
Você pode ver o código enviado em Merge Request para outro repositório ou no repositório atual.
exemplo http://demo.gitlab.com/diaspora/diaspora/commits/master
Então você pode adicionar comentários às alterações do arquivo submetido (botão Reply) ou a todo o commit
exemplo http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c
A comunicação resultante é a revisão do código . No entanto, eu pessoalmente recomendo fazer a revisão do código em um PC com comunicação face a face sempre que possível e usar ferramentas para registrar os resultados ou quando mais formalidade for necessária.
Para um arquivo de revisão que tem muitos commits, por exemplo, http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md olhe para ele usando blamepara entender quem fez o quê. No entanto, nesta visualização, não há opção de comunicar e adicionar comentários. Eu recomendaria apenas adicionar alterações como comentários neste caso.
fonte
Sim. Solicitações de mesclagem são como as revisões de pares são realizadas.
Deve haver uma guia 'diff' que mostrará as alterações de todos os commits (mencionados aqui: http://youtu.be/DyAX8ws5OIc?t=3m2s ).
O vídeo também explica como pode ser usado para revisão por pares.
fonte
O caso de uso normal de revisões de código é revisar o código em um branch antes de fundir no master ou similar. Eu tenho uma situação em que desenvolvi um projeto e quero que todo o código seja revisado por todos da equipe.
O que eu fiz foi:
Verifique o primeiro commit, faça uma mudança nele, faça um commit e push
Verifique o último commit, faça uma mudança nele, faça um commit e push
No GitLab / GitHub, crie uma solicitação pull
Funciona para mim!
fonte
git br --delete --force origin FIRST_COMMIT LAST_COMMIT; git br --delete --force FIRST_COMMIT LAST_COMMIT
). Você pode usar um branch off master diferente para conter as alterações ou criar problemas separados manualmente. E depois crie um ou mais branches (por exemplo, um por problema) se houver muito feedback.