Repository navigation
feat: automates updating the list of contributors - #314
LorhanSohaky wants to merge 2 commits into
Conversation
|
Kudos, SonarCloud Quality Gate passed!
|
pedrosancao
left a comment
pedrosancao
left a comment
There was a problem hiding this comment.
O teste que está no link no comentário na issue 302 duplicou a foto de várias pessoas e como cada imagem ocupa um bom espaço no readme gostaria de manter essa seção o mais enxuta possível, faz sentido? Seria melhor se pudesse filtrar, para manter os devidos destaques para os autores sem duplicar.
A action separa colaboradores e contribuidores, mas no momento não fazemos essa distinção aqui. Ela também usa o nome das pessoas ao invés do usuário, o que tem mais chances de "quebrar" o layout da tabela, é possível configurar isso?
Importante também fazer "squash" dos commits para manter o histórico mais limpo e o que tem sido solicitado em PRs anteriores.
|
Acho que o que você chama de duplicada é poq eu tinha mantido o readme antigo em baixo |
|
Creio que o que ele chama de colaboradores são as pessoa da org que atuam diretamente no projeto |
|
Sobre o squash do commit dá para eu fazer, mas também é possível na hora do merge |
|
Sobre o ponto do nome de usuário eu abri uma issue para ver se é possível fazer isso |
|
@LorhanSohaky algumas considerações:
Vi que atualizou o exemplo, lá você removeu a seção "Autores" é algo que não acho legal de fazer, com a seção mantida vão aparecer fotos duplicadas, e isso acontece no exemplo onde a sua foto aparece duplicada.
No exemplo está separado você como Colaborador (no contexto do seu fork), que seria o equivalente à seção "Autores" existente, mas se usarmos o placeholder para fazer a substituição o resultado vai ser diferente do que existe, e até pelo título da PR não acho que seja intenção alterar autores/colaboradores.
Eu ainda prefiro que seja feito antes do merge, pois a pessoa que faz merge geralmente não é o autor, e o autor tem mais conhecimento para criar uma mensagem de commit com as informações necessárias. PS: Por favor evite várias respostas seguidas, use a opção de editar. O GitHub envia e-mails para cada respostas (dependendo da configuração de cada um) e acaba gerando spam. Ontem mesmo uma pessoa comentou pedindo para parar de receber os e-mails, é outro contexto mas é apenas um exemplo. |
|
@pedrosancao , eu tinha separado para dar a noção de thread |
This branch was successfully deployed
2 participants
No description provided.