A instrutora não deveria ter feito as correções em uma branch fix/bugfix em vez de feat? Pois não se trata de novas funcionalidades e melhorias, e sim correções apontados no codeReview.
Você está vendo a versão anterior da nova experiência da Alura que estamos preparando para você. Em breve, ela ganha uma identidade visual novinha totalmente pensada em potencializar seus estudos!
A instrutora não deveria ter feito as correções em uma branch fix/bugfix em vez de feat? Pois não se trata de novas funcionalidades e melhorias, e sim correções apontados no codeReview.
Olá, Artur, como vai?
Sua observação faz sentido considerando uma convenção bastante comum em projetos que utilizam nomes de branches para representar o tipo de alteração. Uma branch com feat costuma estar associada à implementação de uma nova funcionalidade, enquanto fix ou bugfix costuma ser utilizada quando o objetivo principal é corrigir um comportamento ou problema existente. Nesse exercício, as alterações feitas durante o code review estão corrigindo pontos da implementação que já foi enviada na pull request, então fix seria uma nomenclatura coerente para representar esse trabalho.
Ao mesmo tempo, é importante separar a finalidade da branch da finalidade da alteração feita durante a revisão. O nome da branch original pode continuar sendo feat, porque ela foi criada para desenvolver a nova seção de perfil do formulário. Durante o code review, podem surgir ajustes necessários nessa mesma implementação, como adicionar required, corrigir um value ou remover algum detalhe incorreto. Esses ajustes não transformam a branch original em uma branch de correção, já que continuam fazendo parte da entrega daquela funcionalidade. Em um fluxo diferente, em que a correção fosse feita separadamente depois que a funcionalidade já estivesse integrada, uma fix faria ainda mais sentido.
Na sua visão, em um projeto real você manteria essas correções na mesma branch da funcionalidade ou criaria uma branch específica de fix para cada problema encontrado durante o code review?
Continue compartilhando esse tipo de observação, porque analisar não apenas o código, mas também o fluxo de trabalho com Git, ajuda bastante a entender como essas práticas são aplicadas no dia a dia. O fórum está à disposição para continuar essa discussão.
Para saber mais:
Conte com o apoio da comunidade Alura na sua jornada. Abraços e bons estudos!