okfn-brasil / querido-diario

📰 Diários oficiais brasileiros acessíveis a todos | 📰 Brazilian government gazettes, accessible to everyone.
https://queridodiario.ok.org.br/
MIT License
1.04k stars 384 forks source link

[Revisão retroativa] Imprensa Oficial #1125

Open claromes opened 3 months ago

claromes commented 3 months ago

Layout do site publicador de diários oficiais

Marque apenas um dos itens a seguir:

Código da(s) spider(s)

Testes

Verificações

Descrição

Issue: #951

Lista:

Comentários

trevineju commented 2 months ago

@claromes, obrigada por estar fazendo essa tarefa!

As coisas importantes a ser feitas são:

  1. Testar a classe base, que depende de alguns municípios
  2. Para os casos que ainda tem diários, mas não são mais atuais, adicionar o end_date e modificar o nome.
  3. Para os casos que o site não mais existe ou não tem mais diários, excluir o arquivo do repositório. (A gente não tem por quê manter o código de raspagem de um site que não tem mais diários)

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:
The files between the years 2019 and 2022 are available using the Imprensa Oficial system. This city now uses the BR Transparência system, which covers other cities. The Base is not developed.

O fato do raspador estar com start_date, end_date e herdar a base ImprensaOficialSpider já evidencia a informação da primeira linha, então é redundante. E, quanto a segunda, caberia mais abrir uma issue para o novo padrão que achou e listar todos os casos que já viu fazer parte do padrão.

Com isso, pode tirar todas as demais cidades dessa PR. E justando esse pouquinho, já estaria pronta pra revisão.

💯

trevineju commented 2 months ago

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 ;)

claromes commented 1 month ago

@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. Pronto para revisão! \o/