Atualizados 2 métodos, criados 3 métodos#1
Atualizados 2 métodos, criados 3 métodos#1JoaoPauloJorgeDeOliveira wants to merge 2 commits intoWittmannF:masterfrom
Conversation
|
Fala @JoaoPauloJorgeDeOliveira ! Eu não esqueci de suas excelentes contribuições! Só estou pensando em como fazer elas ao mesmo tempo que mantenho o arquivo original do curso intacto para que não confundam os alunos. Uma ideia que tive aqui seria passar essas sugestões de melhoramentos para um segundo arquivo, talvez poderíamos chamar ele de |
|
Boa noite, @WittmannF ! |
|
Faz sentido! Meu único receito é aceitar pull requests que talvez "quebrem"
o código. Quanto maior o código, maior também deverá ser a manutenção..
…On Thu, May 7, 2020 at 9:15 PM JoaoPauloJorgeDeOliveira < ***@***.***> wrote:
Boa noite, @WittmannF <https://github.com/WittmannF> !
Pessoalmente, eu não acho que ficaria confuso manter apenas 1 arquivo.
Meu primeiro commit era apenas acrescentando métodos que você já explica
no curso. Eles só não estavam aqui no código.
Já o segundo realmente implementa 3 métodos novos. Porém, acho que o aluno
que estiver fazendo o curso simplesmente se concentraria nos métodos que
você explica nas aulas e ignoraria os novos.
Se eu estivesse fazendo o curso neste momento, acho que eu ficaria com
mais dúvidas se houvessem 2 arquivos (e eu não soubesse qual deles usar) do
que se houvesse um único arquivo com uma parte não abordada nas aulas.
Além disso, acho que há a opção de colocar um comentário escrito (ou um
vídeo de 2 minutos) no curso apenas avisando que o código foi atualizado.
Porém, fique a vontade para proceder como achar melhor. 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLXW7UJZENACLILBK4OC6TRQNFKLANCNFSM4J5GCJCA>
.
|
Após fazer o curso na Udemy, percebi que o código no GitHub não era o mais atualizado.
Assim, atualizei com os 2 métodos faltantes.
Além disso, criei 3 novos métodos, visando: