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.11k stars 411 forks source link

Cria base para municípios que usam solução da Portalgov #1253

Closed slfabio closed 2 months ago

slfabio commented 3 months ago

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)

Testes

Verificações

Descrição

Implementa base para a solução da Portalgov, utilizada pelos municípios de São João da Barra e Varre-Sai, conforme issue: #1246 Obs. embora no site de São Joaão da Barra tenha três anteriores a 15/07/2013, todas elas se referem a edições mais recentes e estão duplicadas no site, por isso a start_date foi definido para 15/07/2013.

slfabio commented 2 months ago

@slfabio, valeu! Tô aprovando as modificações. Poderia fazer squash nos 3 commits pra ficar apenas um, sem os commits de correção? Se precisar de ajuda, pode falar. Ou também posso fazer por aqui se preferir

Bom dia, @ogecece , tentei fazer o squash. No repositório local só aparece um commit no log agora, mas aqui no PR está com todo histórico de commits, não sei se deu certo.

trevineju commented 2 months ago

oi, oi! Como @ogecece já aprovou e faltava só ajustar o histórico vou ajudar pra já ir colocando os raspadores pra rodar, tudo bem?

Corrigi o histórico, mas deixo print pra discutir a solução.

Fiquei com a impressão que você tinha feito certo local, @slfabio, só que tinha que dar git push com -f para forçar que suas mudanças locais sobreescrevessem as que estão na PR, evitando o merge entre a local e a remota que ocorreu em 7669e3f.

Screenshot from 2024-09-06 08-50-53

Desfiz o merge, fiz o squash (befda9f5ced0cc938c4aa5bba493df0c6577c171), subi com git push -f e foi isso!

Valeu!

slfabio commented 2 months ago

Muito obrigado, @trevineju !