Adicionar novas cidades de sistemas replicáveis (3/3)

[trevineju]

Tarefa aberta para o workshop na EACH-USP.

Exemplo de raspador instar funcional: sp_barao_de_antonina.py Exemplo de raspador doem funcional: ba_acajutiba.py

• [ ] PR 1 (doem) Santa Rita de Cássia BA: https://doem.org.br/ba/santaritadecassia (instar) Aparecida SP: https://www.aparecida.sp.gov.br/portal/diario-oficial (instar) Valparaiso SP: https://www.valparaiso.sp.gov.br/portal/diario-oficial

• [x] PR 2 (doem) Sátiro Dias BA: https://doem.org.br/ba/satirodias (doem) Tapiramutá BA: https://doem.org.br/ba/tapiramuta (doem) Tamboara PR: https://doem.org.br/pr/tamboara

• [x] PR 3 (doem) Itamaraju BA: http://doem.org.br/ba/itamaraju (instar) Botucatu SP: https://www.botucatu.sp.gov.br/portal/diario-oficial (instar) Ourinhos SP: https://www.ourinhos.sp.gov.br/portal/diario-oficial

• [x] PR 4 (instar) Cachoeira do Sul RS: https://www.cachoeiradosul.rs.gov.br/portal/diario-oficial (instar) Januária MG: https://www.januaria.mg.gov.br/portal/diario-oficial (instar) Monte Mor SP: https://www.montemor.sp.gov.br/portal/diario-oficial

[MatmaK]

Vou fazer esse!

[MatmaK]

@trevineju uma dúvida, preciso criar uma branch nova antes de fazer o pr? Tem algum padrão pro nome das branchs e pras mensagens de commits?

Ah, e recomenda criar um commit pra cada raspador ou, por serem simples, posso criar um commit pra cada pr?

[trevineju]

Oi, @MatmaK voa aí 🚀

@trevineju uma dúvida, preciso criar uma branch nova antes de fazer o pr?

Isso, cada PR demanda uma branch, vc vai acabar criando 4 branchs para concluir essa issue

Tem algum padrão pro nome das branchs?

Não tem exatamente um padrão pra nomear branchs. A boa prática é dar um nome que se relacione a tarefa que a branch pretender fazer (no caso, poderia ser algo como novascidades ou algo assim)

e pras mensagens de commits? Ah, e recomenda criar um commit pra cada raspador ou, por serem simples, posso criar um commit pra cada pr?

Também não precisa seguir um padrão pras mensagens de commits (de referência, o pessoal costuma usar "Add: raspador pra tal lugar"), mas assim como nomear branchs, o ideal é escrever uma mensagem que se relacione ao código sendo enviado no commit. Esse cuidado já é suficiente.

Por ser simples, pode sim enviar um commit por PR, enviando o único commit como um "pacote de cidades" (e aí a mensagem de commit informa que adiciona vários)

[trevineju]

@MatmaK, vi que tá voando!! boa demais em gerir as branchs bem, agora!

Sei que perdeu o finzinho do workshop, mas as issues foram abertas propondo que a PR 1 tenha só 1 cidade pq o pessoal tava aprendendo ainda (pratica com uma antes de enviar 3). Mas agora no seu caso como claramente tá conseguindo lidar, acho um desperdício vc se esforçar pra mandar só 1 cidade. Posso editar a issue para ser, tb, 3 ali na PR 1?

[MatmaK]

@MatmaK, vi que tá voando!! boa demais em gerir as branchs bem, agora!

Sei que perdeu o finzinho do workshop, mas as issues foram abertas propondo que a PR 1 tenha só 1 cidade pq o pessoal tava aprendendo ainda (pratica com uma antes de enviar 3). Mas agora no seu caso como claramente tá conseguindo lidar, acho um desperdício vc se esforçar pra mandar só 1 cidade. Posso editar a issue para ser, tb, 3 ali na PR 1?

Pois é, comecei do final kkkkk mas pode adicionar sim, @trevineju, por favor. Ahh, se tiver alguma cidade que já sabe que precisa criar o raspador do zero e puder me mandar, por favor. Queria tentar fazer.

[trevineju]

editada! @MatmaK

Só pra avisar que ainda não realmente revisei, só olhei por cima suas PRs. Vou maratonar revisões hoje a tarde. E aí, mais tarde, também indico uma issue de um raspador do zero -

Obrigada demais por estar contribuindo!!