brutils-python icon indicating copy to clipboard operation
brutils-python copied to clipboard

feat:retorna codigo do ibge do nome do municipio e da uf

Open CarduCaldeira opened this issue 1 year ago • 1 comments

Descrição

Implementa uma função que recebe o nome do municipio e a sigla da UF e retorna seu código do IBGE. @mateusoliveira43, @pedromendesjr, @nettorobson

Mudanças Propostas

Checklist de Revisão

  • [x] Eu li o Contributing.md
  • [x] Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
  • [ ] Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
  • [ ] A documentação em português foi atualizada ou criada, se necessário.
  • [ ] Se feita a documentação, a atualização do arquivo em inglês.
  • [x] Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
  • [ ] O código segue as diretrizes de estilo e padrões de codificação do projeto.
  • [x] Todos os testes passam. Instruções
  • [x] O Pull Request foi testado localmente. Instruções
  • [x] Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Estamos na dúvida na utilização de um arquivo json para leitura dos códigos referentes ao municípios (caso implementado) ou utilização de uma API externa para obter tais códigos.

Uma sugestão de desenvolvimento para melhoria seria a criação de um processo de automação para atualização do json

Issue Relacionada

Closes #399

CarduCaldeira avatar Sep 13 '24 20:09 CarduCaldeira

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.67%. Comparing base (a46eafe) to head (c88460f). Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #411      +/-   ##
==========================================
- Coverage   99.83%   99.67%   -0.16%     
==========================================
  Files          21       21              
  Lines         591      612      +21     
==========================================
+ Hits          590      610      +20     
- Misses          1        2       +1     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Sep 13 '24 20:09 codecov[bot]

@camilamaia obrigado pela ajuda, em breve aplicarei as mudanças!

CarduCaldeira avatar Sep 28 '24 11:09 CarduCaldeira

O problema de coverage tá dando num arquivo nada relacionado com as tuas mudanças 🤷‍♀️ Então vou mergear e depois eu vejo esse erro específico, beleza?

image

camilamaia avatar Dec 19 '24 13:12 camilamaia