-
-
Notifications
You must be signed in to change notification settings - Fork 382
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
Adiciona vários spider de MG - Barbacena | Carmo do Rio Claro | Onça do Pitangui | Juatuba #1060
base: main
Are you sure you want to change the base?
Conversation
85eeef7
to
f2a59a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parabéns pela primeira contribuição, @lucioerlan!
Precisei fazer só alguns ajustes, que você pode conferir no commit f2a59a3.
@lucioerlan, fui testar e vi que o raspador para Barbacena-MG está dando muito erro 403 e 404 de arquivo que existe no site: barbacena-v1.log Você poderia dar uma olhada nisso? Os outros raspadores estão certos ;) |
Found other issues after my first review
obrigado @trevineju, claro, estou verificando aqui ;) |
@lucioerlan, obrigada por ver! Me ocorreu agora também sugerir que, como 3 dos 4 raspadores estão revisados, daria pra tirar mg_barbacena daqui, eu aprovo a PR e colocamos as três cidades validadas em produção (não tem pq a gente deixar código bom parado, sabe?). E aí você abre outra PR com mg_barbacena, talvez como rascunho, e vai trabalhando nela. O que você acha? |
Checklist - Novo spider
start_date
eend_date
definidos) ao menos uma vez e os dados retornados estavam corretos.log_count/ERROR
igual a zero).start_date
no seu spider com a data do Diário Oficial mais antigo disponível na página da cidade.Descrição
Adicionando vários spiders para o estado de Minas Gerais(MG) - Barbacena | Carmo do Rio Claro | Onça do Pitangui | Juatuba
Informações Complementares:
Barbacena, MG: 3105608
Carmo do Rio Claro, MG: 3114402
Onça do Pitangui, MG: 3151404
Juatuba, MG: 3136652