-
-
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
[Mapeamento] Lista de municípios que usam INSTAR #1088
Comments
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? |
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 be5a58e 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! 😄 |
a qualidade da "lição de casa" foi super bem feita (e além!) pelo visto, @jreagle! hahahah
é 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. |
Claro! Será uma honra e um baita aprendizado! 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
The text was updated successfully, but these errors were encountered: