4 - Code review

[lidimayra]

@gagres @marqueslu

O repositório de cada um de vocês recebeu um pull request meu! 😛

Owner	PR
@gagres	link
@marqueslu	link

Porééém... eu fui um tico relaxada... eu deveria alterar apenas o que fosse conveniente, mas fiquei mexendo nos seus arquivos e acabei mudando coisa que não devia! 😜 😱

Vocês irão avaliar os pull requests. Usando a ferramenta de revisão de código do GitHub, analisem cada uma das linhas que foi alterada por mim.

Cada vez que vocês encontrarem algo que queiram comentar, xingar, elogiar ou questionar, cliquem na linha correspondente para referenciá-la e deixe um comentário lá.

Vocês terão a opção de deixar comentários avulsos (Add Single Comment) ou acumulá-los em uma única revisão (Start a review).

Eu recomendaria o segundo, pra que vocês possam empacotar tudo de uma vez, mas sintam-se à vontade pra usar o que preferirem. Podem até brincar com ambos, inclusive!

Depois que tiverem feito todos os comentários que achar neemcessários, vocês irão finalizar o review. Neste momento vocês podem:

• Comentar (caso queiram apenas fazer alguma observação simples que não seja crítica para a aceitação do meu PR)
• Aprovar (se o meu PR estiver perfeitinho, sem problema nenhum)
• Exigir Alterações (caso achem que o meu pull request só possa ser aceito se eu fizer as correções pedidas por vocês nos comentários anteriores)

Podem mandar bala sem dó! 😄

Obs: Não façam o merge do PR ainda, esta tarefa se limita apenas ao review

---

• @gagres
• @marqueslu

[lidimayra]

@marqueslu: meu PR foi mergeado com os erros... agora a sua master contém coisas que não devia! 😄
Sugiro que faça o revert abrindo um outro pull request clicando no botão revert:

Desta forma você voltará a branch master ao estado em que se encontrava antes do merge 😉

[gagres]

Feito Lidi!!

[lidimayra]

Booooa, @gagres!!!
Fiz as correções que você pediu no PR! Aguardo novo review! 😄

[marqueslu]

Feito Lidi!!

[lidimayra]

Excelente, @marqueslu!! Comentei lá no PR! 😄

[lidimayra]

Show, @marqueslu!!!

Agora que o revert já foi mergeado, abri um novo PR contendo as mesmas alterações para darmos continuidade à tarefa! Qualquer coisa, já sabe, só chamar! 💜

[lidimayra]

@gagres, tarefa concluída! Sinta-se à vontade pra dar o start na #5 quando quiser! 🎉