Create Municipality Method by jaimenunes · Pull Request #412

Create Municipality Method by jaimenunes · Pull Request #412 · brazilian-utils/python

Conversation

Descrição

Criação de uma nova funcionalidade para a obtenção de uma tupla contendo (município, UF) dado um código válido do IBGE.

Mudanças Propostas

Criação de funcionalidade

Checklist de Revisão

Eu li o Contributing.md
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.
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.
Todos os testes passam. Instruções
O Pull Request foi testado localmente. Instruções
Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Issue Relacionada

Closes #398

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.80%. Comparing base (4af1cdc) to head (d701f1f).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   99.78%   99.80%   +0.01%     
==========================================
  Files          18       19       +1     
  Lines         472      521      +49     
==========================================
+ Hits          471      520      +49     
  Misses          1        1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jaimenunes ficou incrível, muito obrigada pela contribuição! 🌟 Pelo que percebi está faltando apenas a entrada no CHANGELOG.md, e alguns ajustes nos testes. Os avisos do codecov podem ajudar, nos avise se precisar de alguma ajuda!!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa, @jaimenunes um PR de rexxxpeito!! 🚀 Ficou mesmo só aquele detalhe no arquivo CHANGELOG.md que não consegui ajustar por aqui, senão já era merge certeiro! Obrigada pela contribuição!! 🥇

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Booooa @jaimenunes! Eu só fiz duas minis sugestões que já vou comitar e mergear mesmo. Valeu pela contribuição!!