Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Oct 24, 2023

Conversation

MatmaK
Copy link
Contributor

@MatmaK MatmaK commented Oct 23, 2023

Checklist - Novo spider

  • Você executou uma extração completa do spider localmente e os dados retornados estavam corretos.
  • Você executou uma extração por período (start_date e end_date definidos) ao menos uma vez e os dados retornados estavam corretos.
  • Você verificou que não existe nenhum erro nos logs (log_count/ERROR igual a zero).
  • 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.
  • 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

Copy link
Member

@trevineju trevineju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MatmaK está excelente a PR! Adicionei só um ajuste que vou eu mesma acolher pq foi bobeirinha (:

data_collection/gazette/spiders/sp/sp_monte_mor.py Outdated Show resolved Hide resolved
@trevineju trevineju added the hacktoberfest-accepted Pull Requests aprovados na Hacktoberfest label Oct 24, 2023
@trevineju
Copy link
Member

@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
Copy link
Contributor Author

MatmaK commented Oct 24, 2023

@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
Copy link
Member

trevineju commented Oct 24, 2023

@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:

  1. git branch (para verificar se você está nesta branch novas-cidades)
  2. se não estiver: git checkout novas-cidades (passará a estar na branch certa). Se já estiver, ignora esse passo.
  3. 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.)
  4. git status (para confirmar o que disse acima, que as modificações seguem na sua máquina local de fato)
  5. 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):

  1. git checkout main
  2. 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)
  3. git checkout -b novas-cidades-pr2
  4. 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
Copy link
Member

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

@MatmaK
Copy link
Contributor Author

MatmaK commented Oct 24, 2023

@trevineju acho que deu certo. Obrigado!

Copy link
Member

@trevineju trevineju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ficou excelente, @MatmaK! PR revisada e aprovada!! ❤️ Obrigada pela contribuição! 🎉

habilitei estes novos raspadores em produção, os dados raspados estarão no ar a partir de amanhã

@trevineju trevineju merged commit 4dd46b2 into okfn-brasil:main Oct 24, 2023
1 check passed
@trevineju trevineju linked an issue Oct 26, 2023 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull Requests aprovados na Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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