tabnews.com.br
tabnews.com.br copied to clipboard
filipedeschamps
style(footer/index.js): align the tabnews icon with the footer text
🤔 Por que você está abrindo esse Pull Request?
Um simples ajuste no alinhamento do ícone do rodapé com o texto, algo que estava me incomodando faz tempo 😅️
🧐 Descreva sua solução:
Basicamente foi adicionado display: 'flex justifyContent: 'center'. Conforme imagem abaixo:
@rodrigoKulb is attempting to deploy a commit to the TabNews Team on Vercel.
A member of the Team first needs to authorize it.
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
Legal pessoal será que se bater 6 aprovações a PR vai ser aprovada na Vercel sem aval do Filipedeschamps ? Faltam somente 2 aprovações!
At least 6 approving reviews are required by reviewers with write access.
@rodrigoKulb acredito que isso só aconteceria se fossem 6 aprovações de pessoas com acesso lá na Vercel, como o @aprendendofelipe.
Não vale nada testar, vou fazer um review aqui.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated |
|---|---|---|---|
| tabnews | ✅ Ready (Inspect) | Visit Preview | Dec 8, 2022 at 6:28PM (UTC) |
Legal pessoal será que se bater 6 aprovações a PR vai ser aprovada na Vercel sem aval do Filipedeschamps ? Faltam somente 2 aprovações!
A única coisa que vai acontecer com as 6 aprovações é liberar o merge na main, mas não o deploy na Vercel. Só que agora com o repositório público os acessos foram revogados, então ninguém vai conseguir fazer o merge na main. Fazer o deploy na Vercel é independente disso, basta ter acesso na Vercel
Como que o deploy foi feito no ambiente de Preview pelo commit do @rodrigoKulb?
Como que o deploy foi feito no ambiente de Preview pelo commit do @rodrigoKulb?
Eu fiz o deploy, mas em seguida já excluí, fiz o rebase e de novo fiz o deploy
@aprendendofelipe quero propor uma coisa: você topa ir fazendo o merge e deploy destes PRs sem passar por minha revisão ou aprovação? Só peço que tome nota para eu conseguir destacar eles mais facilmente no próximo changelog e já antecipar eles na lista de tarefas da próxima Milestone.
No meu changelog retroativo eu tenho isso aqui anotado que precisa ser publicamente comemorado (e incluído numa Milestone):
https://github.com/filipedeschamps/tabnews.com.br/pull/823 https://github.com/filipedeschamps/tabnews.com.br/pull/811 https://github.com/filipedeschamps/tabnews.com.br/pull/882 https://github.com/filipedeschamps/tabnews.com.br/pull/894 https://github.com/filipedeschamps/tabnews.com.br/pull/896
https://github.com/filipedeschamps/tabnews.com.br/pull/976 https://github.com/filipedeschamps/tabnews.com.br/pull/985 https://github.com/filipedeschamps/tabnews.com.br/pull/988 https://github.com/filipedeschamps/tabnews.com.br/pull/1001
https://github.com/filipedeschamps/tabnews.com.br/pull/1022 https://github.com/filipedeschamps/tabnews.com.br/pull/1060
https://github.com/filipedeschamps/tabnews.com.br/pull/1045 https://github.com/filipedeschamps/tabnews.com.br/pull/1061
https://github.com/filipedeschamps/tabnews.com.br/pull/1064 https://github.com/filipedeschamps/tabnews.com.br/pull/1067
https://github.com/filipedeschamps/tabnews.com.br/pull/1070 https://github.com/filipedeschamps/tabnews.com.br/pull/1081
@aprendendofelipe quero propor uma coisa: você topa ir fazendo o merge e deploy destes PRs sem passar por minha revisão ou aprovação? Só peço que tome nota para eu conseguir destacar eles mais facilmente no próximo changelog e já antecipar eles na lista de tarefas da próxima Milestone.
Opa, claro! Posso ver o que já está pronto e fazer o merge. Só que precisa mudar as permissões da branch main no GitHub.
Acho que é só diminuir as aprovações de 6 para 1. Dessa forma os meus PRs precisarão da aprovação de pelo menos um outro colaborador, mas os demais eu já consigo fazer o merge se eu mesmo aprovar.
E isso é uma ótima oportunidade pra gente testar o fluxo com a branch production. Que tal? Nesse caso é só fazer a mesma configuração nessa branch que vou criar agora.
Acho que é só diminuir as aprovações de 6 para 1. Dessa forma os meus PRs precisarão da aprovação de pelo menos um outro colaborador, mas os demais eu já consigo fazer o merge se eu mesmo aprovar.
Feito! Agora só é preciso e 1 approval 🤝
E isso é uma ótima oportunidade pra gente testar o fluxo com a branch production. Que tal? Nesse caso é só fazer a mesma configuração nessa branch que vou criar agora.
Sugiro fazermos aquele call para discutir o fluxo, ou discutir async mesmo, porque eu ainda fiquei com dúvidas sobre os tradeoffs dessa estratégia 👍
Sugiro fazermos aquele call para discutir o fluxo, ou discutir async mesmo, porque eu ainda fiquei com dúvidas sobre os tradeoffs dessa estratégia 👍
@filipedeschamps o que acha de organizarmos esse call lá no servidor do TabNews no discord? Lá tem canais privados especialmente feitos isso. Dá também pra separar esses canais por prioridade de hierarquia, ou seja, pessoas com cargos menores não podem entrar nesses canais que são para cargos maiores, perfeito para o nosso caso.
Inclusive, você já está nele: https://discord.gg/Yu9N2kFNQ5
Feito! Agora só é preciso e 1 approval 🤝
Tem mais alguma coisa para configurar, pois não habilitou o botão de merge.
Sugiro fazermos aquele call para discutir o fluxo, ou discutir async mesmo, porque eu ainda fiquei com dúvidas sobre os tradeoffs dessa estratégia 👍
Bora fazer essa call 🚀
Tem mais alguma coisa para configurar, pois não habilitou o botão de merge.
Interessante que no PR aberto hoje o botão de merge está disponível para mim. Será que só vai valer para os novos PRs? E se eu fechar e reabrir esse? Vou testar
Acho que eu sei o que aconteceu, quando eu reduzi para 1 vote, eu habilitei isso aqui:
Mas isso vai te travar @aprendendofelipe então desabilitei 👍
Acho que eu sei o que aconteceu, quando eu reduzi para 1 vote, eu habilitei isso aqui:
Mas isso vai te travar @aprendendofelipe então desabilitei 👍
Era isso! Agora está habilitado o botão 👍🎉
Acho que não tem mesmo necessidade disso, pois os PRs abertos por mim já vão exigir a sua aprovação antes do merge.