wiz-fatec / api-2BD

Repositório dedicado ao versionamento do projeto de API do segundo semestre de Banco de Dados.
3 stars 1 forks source link

🧑‍💻 Change getSubmit HashMap to Set #55

Closed ChavesVini closed 11 months ago

MarcyLeite commented 11 months ago

Algumas notas pra vocês, acho que podia usar kebab-case pra separar o nome da branch connect-database. Como eu disse pessoalmente, mudanças de escopo minimo no código não precisam ser feitas em outra branch, faz na DEV msm, só me mantem avisado. Isso dito, se o escopo for maior e precisar de PR, nomear a branch de acordo com a mudança que esta sendo feita, tem bons exemplos de nomenclatura de branch. Nesse caso, nem a mudança que o @ChavesVini fez 38c1104, nem a do @PedroBozzano fez 5df67d7 tem uma relação clara com connect-database. Isso dito, se as duas mudanças fossem grande, a do Vini teria que ser em uma branch (ex: fix/hash-to-set) e a do Pedro (ex: feat/to-do-database)

Dito isso, eu vou aprovar a branch, mas na dev vou reverter as mudanças que o Pedro fez, e ai ele pode continuar trabalhando na propria branch sem afetar a dev.