search

pyladies-brazil / br-pyladies-pelican

Site PyLadies Brasil usando Pelican
http://brasil.pyladies.com/
MIT License
69 stars 104 forks source link

[Documentação] Adicionar explicação sobre como atualizar o fork #423

Closed giovana-morais closed 3 years ago

giovana-morais commented 3 years ago

À medida que a gente muda algumas coisas importantes, como foi a remoção do arquivo de ladies.yml ou a atualização da versão da bibliotecas que usamos, alguns PRs podem vir desatualizados e causar um retrabalho na hora da revisão.

Acho que é importante adicionar na documentação um pequeno trecho pra deixar o fork apontando pro repositório original e lembrando de sempre atualizar o próprio branch antes de fazer a contribuição. :wink:

alynnefs commented 3 years ago

A ideia é mudar no README.md ou no CONTRIBUTING.md? Acabei de seguir esses passos e deram certo. Tendo isso definido, posso adicionar a explicação (ou deixo livre pra quem quiser treinar PR)

giovana-morais commented 3 years ago

@alynnefs a ideia seria adicionar ao CONTRIBUTING. talvez colocar como um subtópico ou antes de "preparando ambiente local" ou mais pro fim. não sei exatamente onde, estou aberta a sugestões hahaha

alynnefs commented 3 years ago

@giovana-morais acho que antes de "preparando ambiente local" é um bom local

obs: dei uma lida no documento e percebi que nunca fiz uma branch específica, sempre usei a principal. Foi mal kkkk

alynnefs commented 3 years ago

Estava testando as confirmações do PR #424 e percebi que o link "Adicionar ladies" está quebrado, pois o texto foi removido em algum outro PR. Posso remover nesse PR ou abro outro?

giovana-morais commented 3 years ago

pode remover nesse mesmo. é só um link, acho que não vale a pena abrir um PR só pra isso.

alynnefs commented 3 years ago

PR atualizado removendo o link 😃

giovana-morais commented 3 years ago

obrigada pelo PR! vou fechar essa issue aqui (: