Closed jreagle closed 3 weeks ago
between_collecttest_to_caseara_20240101_20240131.csv between_collecttest_to_caseara_20240101_20240131.log full_collecttest_to_caseara_20240927.csv full_collecttest_to_caseara_20240927.log last_collecttest_to_caseara_20240926.csv last_collecttest_to_caseara_20240926.log
between_collecttest_to_itapiratins_20230101_20230131.csv between_collecttest_to_itapiratins_20230101_20230131.log full_collecttest_to_itapiratins_20240927.csv full_collecttest_to_itapiratins_20240927.log last_collecttest_to_itapiratins_20240110.csv last_collecttest_to_itapiratins_20240110.log
between_collecttest_to_miracema_20240101_20240131.log between_collecttest_to_muricilandia_20240101_20240131.csv full_collecttest_to_miracema_20240927.csv full_collecttest_to_miracema_20240927.log last_collecttest_to_miracema_20240926.csv last_collecttest_to_miracema_20240926.log
between_collecttest_to_muricilandia_20240101_20240131.csv between_collecttest_to_muricilandia_20240101_20240131.log full_collecttest_to_muricilandia_20240927.csv full_collecttest_to_muricilandia_20240927.log last_collecttest_to_muricilandia_20240924.csv last_collecttest_to_muricilandia_20240924.log
between_collecttest_to_santa_fe_do_araguaia_20240101_20240131.csv between_collecttest_to_santa_fe_do_araguaia_20240101_20240131.log full_collecttest_to_santa_fe_do_araguaia_20240927.csv full_collecttest_to_santa_fe_do_araguaia_20240927.log last_collecttest_to_santa_fe_do_araguaia_20240924.csv last_collecttest_to_santa_fe_do_araguaia_20240924.log
between_collecttest_to_talisma_20240101_20240131.csv between_collecttest_to_talisma_20240101_20240131.log full_collecttest_to_talisma_20240927.csv full_collecttest_to_talisma_20240927.log last_collecttest_to_talisma_20240912.csv last_collecttest_to_talisma_20240912.log
Esqueci de alguma coisa @trevineju ???
Vou revisar esse
show, @jreagle! Parabéns pela PR! 🎈
Tá com @ddevdan revisar, mas vou deixar só um item de atenção...
1710904 | Itapiratins - TO (Última Edição: 10/01/2024 => "end_date")
Isso aponta que o município parou de usar esse site. Coincidentemente, atualizei a docs oficial com orientações sobre isso ontem mesmo: diários oficiais em diferentes lugares
Basicamente, não poderemos colocar esse raspador em produção enquanto não tivermos o do site atual também. Mas não é um problema o código existir no repositório, só a coleta não vai ser agendada por enquanto.
Deixo a critério de vocês o que decidir, em particular de @ddevdan como revisor. Algumas possibilidades são:
Gente, como que pra dar merge iria precisar atualizar essa PR com a main de qualquer forma, meu pre-commit já vai arrumar o erro de lint. Então, como @ddevdan aprovou, já vou ajustar por aqui, tudo bem? (eu já tava colocando novos raspadores no ar e era mais prático pra mim colocar estes juntos tudo de uma vez, :3 )
Acho que não vai prejudicar o aprendizado de @jreagle visto que erros de lint vão aparecer sempre (heheh), então vai ter mais oportunidades pra treinar.
Deixo o print abaixo para mostrar o arquivo que o linter tava reclamando
Gente, como que pra dar merge iria precisar atualizar essa PR com a main de qualquer forma, meu pre-commit já vai arrumar o erro de lint. Então, como @ddevdan aprovou, já vou ajustar por aqui, tudo bem? (eu já tava colocando novos raspadores no ar e era mais prático pra mim colocar estes juntos tudo de uma vez, :3 )
Acho que não vai prejudicar o aprendizado de @jreagle visto que erros de lint vão aparecer sempre (heheh), então vai ter mais oportunidades pra treinar.
Deixo o print abaixo para mostrar o arquivo que o linter tava reclamando
okay. valeu ju
Parabéns pela contribuição, @jreagle! ❤️ E pela primeira revisão, @ddevdan! ❤️❤️
Vocês foram perfeitos ❤️
Parabéns pela contribuição, @jreagle! ❤️ E pela primeira revisão, @ddevdan! ❤️❤️
Vocês foram perfeitos ❤️
<3<3 obggg
Parabéns pela contribuição, @jreagle! ❤️ E pela primeira revisão, @ddevdan! ❤️❤️
Vocês foram perfeitos ❤️
Pessoal! Desculpa o sumiço, mas tô na área novamente! Aproveitando o feriadão prolongado pra tirar o atraso das contribuições !
Que bom que deu tudo certo por aqui! @ddevdan obrigado p/ disponibilidade e cuidado na revisão dessa minha primeira PR pro QD! A primeira vez a gente nunca esquece, né! 😂
@trevineju gratidão pelos aprendizados e paciência de sempre! 💜
Bora pra próxima! 🚀
AO ABRIR uma Pull Request de um novo raspador (spider), marque com um
X
cada um dos items da checklist abaixo. Caso algum item não seja marcado, JUSTIFIQUE o motivo.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.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
Adiciona 6 raspadores de municípios replicados a partir do padrão BaseDOBR mencionado na https://github.com/okfn-brasil/querido-diario/issues/1090#issuecomment-2372690577: