-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
[Revisão retroativa] Imprensa Oficial #1125
base: main
Are you sure you want to change the base?
Conversation
@claromes, obrigada por estar fazendo essa tarefa! As coisas importantes a ser feitas são:
Pelo que vejo desta PR, ela já cumpre...
... e falta ajustar:
Também vou pedir que retire os comentários de situação. Por exemplo, pegando Itaquara: O fato do raspador estar com Com isso, pode tirar todas as demais cidades dessa PR. E justando esse pouquinho, já estaria pronta pra revisão. 💯 |
Outra coisa... @claromes, você tinha perguntado sobre fazer duas PRs. Tô supondo que uma sendo essa PR nesse ponto que parou e a próxima cobriria as demais cidades, certo? Você até poderia fazer isso, sim. Mas, também, pode só partir pra outra se quiser. Tô dizendo isso pq o complexo você já fez aqui que era validar a classe base, o resto seria só ajustar as classes-filhas e executar testes. Costumo sugerir esse tipo de tarefa super fácil pra pessoas novas na comunidade ou iniciantes querendo contribuir pela primeira vez. Pro seu caso, por ser mais experiente, é mais interessante fazer outra issue, como uma que adiciona outra nova classe base. Isso também é mais interessante pra gente, visto que cada base nos possibilita adicionar dezenas ou centenas de novas cidades. Sua escolha, claro ;) |
@trevineju Obrigada pelos comentários. Fiz as alterações e só testei três cidades que mudaram. Deixei alguns comentários na descrição do PR. |
Layout do site publicador de diários oficiais
Marque apenas um dos itens a seguir:
Código da(s) spider(s)
custom_settings
em meu raspador.Testes
.log
deste teste está anexado na PR..log
e.csv
deste teste estão anexados na PR..log
e.csv
deste teste estão anexados na PR.Gongogi BA:
Última edição:
log_ba_gongogi_2024-05-17.txt
Intervalo (2012-05-01 a 2013-05-01):
log_ba_gongogi_2012-2013.txt
ba_gongogi_2012-2013.csv
Completa:
log_ba_gongogi.txt
ba_gongogi.csv
Itaquara BA:
Última edição:
log_ba_itaquara_2019_2022-01-04.txt (Faz a raspagem, porém, tem um erro no Spidermon)
Intervalo (2020-05-01 a 2021-05-01):
log_ba_itaquara_2019_2020-2021.txt
ba_itaquara_2019_2020-2021.csv
Completa:
log_ba_itaquara_2019.txt
ba_itaquara_2019.csv
Saúde BA:
Última edição:
log_ba_saude_2018_2019-04-12.txt (Faz a raspagem, porém, tem um erro no Spidermon)
Intervalo (2018-03-01 a 2019-03-01):
log_ba_saude_2018-03_2019-03.txt
ba_saude_2018-03_2019-03.csv
Completa:
log_ba_saude_2018.txt
ba_saude_2018.csv
Verificações
.csv
gerados pela minha coleta conforme a documentação não encontrando problemas..log
gerados pela minha coleta conforme a documentação não encontrando problemas.Descrição
Issue: #951
Lista:
ba_conceicao_do_almeida.py
foi deletadoba_itaquara_2019
.ba_saude_2018
.Comentários