Add cidades instar (Cachoeira do sul-RS, Januaria-MG, Monte Mor-SP)

MatmaK commented 11 months ago

Checklist - Novo spider

[X] Você executou uma extração completa do spider localmente e os dados retornados estavam corretos.
[X] Você executou uma extração por período (start_date e end_date definidos) ao menos uma vez e os dados retornados estavam corretos.
[X] Você verificou que não existe nenhum erro nos logs (log_count/ERROR igual a zero).
[X] Você definiu o atributo de classe start_date no seu spider com a data do Diário Oficial mais antigo disponível na página da cidade.
[X] Você garantiu que todos os campos que poderiam ser extraídos foram extraídos de acordo com a documentação.

Descrição

Issue #986 Task 4 Adicionado Spiders das cidades abaixo: Cachoeira do sul-RS, Januaria-MG, Monte Mor-SP

trevineju commented 11 months ago

@MatmaK você vai enviar as demais cidades aqui nesta PR ainda? Acho que você tinha perdido o fim do workshop, mas abri a issue para serem 4 PRs justamente pra ajudar vocês na Hacktoberfest. Mas se você não fizer questão disso, poderia sim mandar tudo em 1 PR só.

Pra avisar que as 3 primeiras cidades estão aprovadíssimas 🥳

MatmaK commented 11 months ago

@MatmaK você vai enviar as demais cidades aqui nesta PR ainda? Acho que você tinha perdido o fim do workshop, mas abri a issue para serem 4 PRs justamente pra ajudar vocês na Hacktoberfest. Mas se você não fizer questão disso, poderia sim mandar tudo em 1 PR só.

Pra avisar que as 3 primeiras cidades estão aprovadíssimas 🥳

@trevineju ia te perguntar sobre isso, eu fiz o ajuste das outras cidades, mas subi na mesma branch aí quando fui abrir um novo pr meio que vinculou a esse já aberto. Teria que criar uma nova branch pra cada pr?

trevineju commented 11 months ago

@trevineju ia te perguntar sobre isso, eu fiz o ajuste das outras cidades, mas subi na mesma branch aí quando fui abrir um novo pr meio que vinculou a esse já aberto. Teria que criar uma nova branch pra cada pr?

Sim, cada branch permite uma única PR. Como vc seguiu adicionando commits nesta branch novas-cidades, o GitHub assume que você está trabalhando nesta PR. Para enviar outra PR, precisa ter outra branch

Como resolver:

Em um terminal aberto no diretório do projeto querido-diario, segue o passo a passo:

git branch (para verificar se você está nesta branch novas-cidades)
se não estiver: git checkout novas-cidades (passará a estar na branch certa). Se já estiver, ignora esse passo.
git reset HEAD~1 (o que isso faz: retrocede em 1 commit o ponteiro que aponta pra cabeça. Se mudar o número, vai desfazer mais commits, mas no seu caso basta tirar um, o último de código 3075a11. Este comando não deve sumir com as modificações, elas vão continuar existindo no seu PC, elas apenas não estarão mais nessa branch.)
git status (para confirmar o que disse acima, que as modificações seguem na sua máquina local de fato)
git push -f (como você retirou um commit desta branch na sua máquina local, precisa "enviar" isso pra esta PR tb. Isso vai atualizar esta página da PR com a remoção)

Até aqui, é esperado que você tenha conseguido corrigir esta branch. Agora pra próxima (como tu já criou uma branch pra enviar esta PR, vou explicar menos a partir de agora):

git checkout main
git pull (sempre atualize sua main antes de abrir uma branch nova para garantir que esteja tão atualizada quanto possível, isso evita muitos problemas)
git checkout -b novas-cidades-pr2
git add <os arquivos de raspadores que estão na sua máquina que foram retirados da branch novas-cidades conferido no passo 4>

trevineju commented 11 months ago

se você tiver problemas com algum desses passos, avisa!

MatmaK commented 11 months ago

@trevineju acho que deu certo. Obrigado!

okfn-brasil / querido-diario