Open trevineju opened 8 months ago
Adiciona alguns municípios INSTAR #1060
Vou trabalhar na cidade São Francisco de Paula - RS
Obrigada, @ddevdan! Será que, a partir de agora, suas PRs contendo esses casos padronizados poderiam ter mais cidades? Talvez umas 5 por vez?
Obrigada, @ddevdan! Será que, a partir de agora, suas PRs contendo esses casos padronizados poderiam ter mais cidades? Talvez umas 5 por vez?
Sim! Tava fazendo a contribuição em uma dinâmica em grupo. Nas próximas fico atento a isso :)
Sim! Tava fazendo a contribuição em uma dinâmica em grupo. Nas próximas fico atento a isso :)
Eu notei vários co-autores! hehehe Costumo sugerir pras pessoas enviar só 1 mesmo, quando acabaram de conhecer o projeto ou estão fazendo a primeira PR. Mas vc já fez algumas e esses casos são muito simples, sabe? Aproveita melhor nosso tempo de desenvolvimento/revisão com mais benefício para o projeto 😄
@trevineju Depois de fazer tarefa de casa de ler a documentação e preparar o ambiente desenvolvimento, estava aqui procurando uma issue pra chamar de minha, conforme tua orientação lá do Discord e depois decidir começar pelos "municípios replicáveis", estava pensando em começar pelos restantes dessa issue da BaseInstar, mas reparei que você já fez todos eles nessa branch instar-restantes e nesse commit https://github.com/okfn-brasil/querido-diario/commit/be5a58e2e4ea3c5e9faa97bd09cd002d6dd2ce96 três meses atrás!
Então, tô só registrando aqui caso outro colega tenha a mesma ideia e também pra te lembrar de atualizar o status por aqui! 😄
mas reparei que você já fez todos eles nessa branch instar-restantes e nesse commit be5a58e três meses atrás!
a qualidade da "lição de casa" foi super bem feita (e além!) pelo visto, @jreagle! hahahah
Então, tô só registrando aqui caso outro colega tenha a mesma ideia e também pra te lembrar de atualizar o status por aqui! 😄
é que três meses atrás estava experimentando criar raspadores padronizados de forma automática e testamos isso em um evento (consistia em uma pessoa contribuidora revisar e testar os raspadores pq pode ter erros de coleta na automação). Exemplos foram: #1185 e #1167 -- ambos os casos foram encontrados problemas e algumas decisões foram tomadas.
Então só pq o commit existe, não significa que o código está bom, tanto é que nem está integrado ao repositório mesmo (por isso essa lista não foi atualizada, pq oficialmente o raspador pra esses municipios não está no projeto). Você quer experimentar revisar? Posso abrir a PR.
Então só pq o commit existe, não significa que o código está bom, tanto é que nem está integrado ao repositório mesmo (por isso essa lista não foi atualizada, pq oficialmente o raspador pra esses municipios não está no projeto). Você quer experimentar revisar? Posso abrir a PR.
Claro! Será uma honra e um baita aprendizado!
Vou precisar comentar na PR também pra você conseguir me atribuir?
Vou precisar comentar na PR também pra você conseguir me atribuir?
Isso!
Sobre essa tarefa
Mapeado em fevereiro de 2024
URL vigente
Tem diários atuais