tabnews.com.br
tabnews.com.br copied to clipboard
feat(evolution-of-tabnews): add evolution of TabNews article to museum page
Conforme sugerido pelo Filipe neste comentário. Eu não sei se era essa a ideia inicial, mas se não for, só avisar pra mim, que faço as alterações necessárias. 🤙
@gabrielsozinho is attempting to deploy a commit to the TabNews Team on Vercel.
A member of the Team first needs to authorize it.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Updated |
---|---|---|---|
tabnews | ✅ Ready (Inspect) | Visit Preview | Nov 24, 2022 at 4:16PM (UTC) |
Hmm, porque do nada a vercel fez o deploy e o github rodou os testes?
Hmm, porque do nada a vercel fez o deploy e o github rodou os testes?
Foi liberado por um membro do time na Vercel 😉
@Rafatcb obrigado pelos toques! Commitei as alterações, está tudo certo?
@Rafatcb obrigado pelos toques! Commitei as alterações, está tudo certo?
Você corrigiu com Find and Replace? Alguns Idéia
perderam o case, ficando ideia
ao invés de Ideia
😅
Você corrigiu com Find and Replace? Alguns Idéia perderam o case, ficando ideia ao invés de Ideia 😅
Caramba, foi exatamente isso. Nem passou pela minha cabeça que isso iria ocorrer.
Da abertura desse PR até hoje, muita coisa mudou no TabNews. Para não precisar ser uma página que é atualizada constantemente, e para continuar fazendo sentido, acho que podemos atualizá-la para "A Evolução do TabNews — da Concepção ao Lançamento" ou algo parecido.
Boa! 👍
Assim, fica em aberto também a possibilidade de incluir mais detalhes em outro PR, como os detalhes que o Filipe comentou.
Isso provavelmente será outro item no museu, mas me levou a pensar se o artigo desse PR não deveria ser um pouco mais impessoal para estar mais aberto a atualizações ou correções futuras. O que acham?
O que acham de hospedar as imagens na pasta public? Eu já vi issues em que as imagens não estão mais aparecendo, mesmo sendo hospedadas pelo próprio GitHub.
Isso provavelmente será outro item no museu, mas me levou a pensar se o artigo desse PR não deveria ser um pouco mais impessoal para estar mais aberto a atualizações ou correções futuras. O que acham?
Eu estava pensando nisso. Na época que eu criei o PR eu pensava que a sugestão do Filipe tinha sido essa (de como está hoje), mas hoje isso não faz tanto sentido.
Também não estou tão disponível para fazer os ajustes necessários, então acho melhor alguém criar um novo PR com os ajustes necessários, não faço questão alguma dos "créditos". (Obs.: o mesmo se aplica ao PR #1210, que está até mais cru que esse aqui, então é ainda melhor outra pessoa criar um novo PR). 👍
Obs.: o mesmo se aplica ao PR https://github.com/filipedeschamps/tabnews.com.br/pull/1210, que está até mais cru que esse aqui, então é ainda melhor outra pessoa criar um novo PR
Eu posso criar um novo PR para o FAQ, tomando como base o seu e os comentários que fiz conforme analisava, só não sei se vou conseguir te deixar como contribuidor nos commits (me parece que o @aprendendofelipe chegou a fazer isso uma vez, mas ainda não parei para entender ao certo como funcionou).
Eu posso criar um novo PR para o FAQ, tomando como base o seu e os comentários que fiz conforme analisava, só não sei se vou conseguir te deixar como contribuidor nos commits (me parece que o @aprendendofelipe chegou a fazer isso uma vez, mas ainda não parei para entender ao certo como funcionou).
Para deixar o @gabrielsozinho como coautor de qualquer commit, basta adicionar Co-authored-by: Gabriel Sozinho <[email protected]>
na última linha da mensagem do commit. Você pode pegar esse dado em qualquer commit dele fazendo um git show <commit>
.
Se for o caso de usar um commit específico dele, sem alterações, aí não fica como coautor, mas sim com a autoria original dele. Nesse caso é só usar o git cherry-pick <commit>
para importar determinado commit na branch que estiver trabalhando.
Se um PR contém todos os commits de outro PR (não tenho certeza se o outro precisa estar aberto), então ao fazer merge de um, o outro também aparece como mesclado. Mas acredito que não vai ser o caso aqui, pois precisaria organizar os commits primeiro.
Gabriel, obrigado pela colaboração! Esse PR está sendo continuado em https://github.com/filipedeschamps/tabnews.com.br/pull/1582, com as modificações propostas nos comentários.