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

PR1 - Cristópolis BA #992

Merged
merged 2 commits into from
Oct 24, 2023
Merged

PR1 - Cristópolis BA #992

merged 2 commits into from
Oct 24, 2023

Conversation

rodrigolyusei
Copy link
Contributor

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: Adicionar novas cidades de sistemas replicáveis (2/3)
Realizei a adição do spider da cidade de Cristópolis citado na PR1.

@trevineju trevineju added the hacktoberfest-accepted Pull Requests aprovados na Hacktoberfest label Oct 24, 2023
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.

@JolidPnake perfeito!! Parabéns pela primeira contribuição! PR aprovada ❤️

Habilitei seu raspador, os dados para Cristópolis-BA estarão no ar a partir de amanhã 🥳

acadasi partyyyyy

Uma dica de GitHub: você não precisa escrever o título da issue, basta referenciá-la com #numero. No seu caso, ficaria "issue #985", só de usar a hashtag o GitHub cria uma referência 😄

@trevineju trevineju merged commit 383b975 into okfn-brasil:main Oct 24, 2023
1 check passed
@trevineju trevineju linked an issue Oct 24, 2023 that may be closed by this pull request
4 tasks
@rodrigolyusei
Copy link
Contributor Author

Entendi! Muito obrigado! 😊
Quando for continuar para fazer as próximas PRs eu faço no mesmo fork que eu criei?

@trevineju
Copy link
Member

Quando for continuar para fazer as próximas PRs eu faço no mesmo fork que eu criei?

isso, mas diria que é melhor fazer a partir de branchs. Se você olhar lá em cima nesta página mesmo, dá pra ver que o "lugar" onde vc adicionou o raspador foi a main.

image

Daí agora, você precisa atualizar a sua main (git pull) e criar uma branch (git checkout -b <nome-da-branch>). Aí seguir fazendo o que já fez aqui (criar os raspadores e enviar commit).

(vi que você entrou no discord da OK. Hoje vou ficar a tarde toda lá, pode aparecer se quiser ajuda ou perguntar algo 😊)

@rodrigolyusei
Copy link
Contributor Author

Beleza! Se precisar eu apareço lá.
Obrigado 👍

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 (2/3)
2 participants