-
-
Notifications
You must be signed in to change notification settings - Fork 416
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 6 novos raspadores #1282
Conversation
Considerando que eu acabei de concluir mais uma revisão da #1284 e verifiquei que essa é a últimas das tasks listadas nela que ainda não foi atribuída, tô pensando em pegar ela pra continuar meu tour pelos Pode ser? |
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.
De forma geral, me pareceu tudo certo com esses spiders de municípios replicados do SpideBase BaseAdiariosV1Spider
. Inclusive, de todos que vi até esse foi o mais gostosinho de revisar pois o servidor PHP por trás me pareceu bem robusto e estável nas respostas.
Enfim, pra mim estão prontos pra entrar em produção! 🚀😉
Show, @jreagle! Fiquei bastante surpresa com os casos que tem muitas edições faltando. |
9438982
to
3856f15
Compare
As spiders adicionadas nesta PR foram criadas com um script criador de código, por isso toda essa PR deve ser revisada em todos os detalhes e também ser testada.
Fica a cargo da pessoa revisora verificar a lista de validações para contribuições.