Skip to content

Conversation

@jaimenunes
Copy link
Contributor

@jaimenunes jaimenunes commented Sep 15, 2024

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
Copy link

codecov bot commented Sep 15, 2024

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:

@jaimenunes jaimenunes marked this pull request as ready for review September 15, 2024 18:19
@jaimenunes jaimenunes requested review from a team as code owners September 15, 2024 18:19
Copy link
Member

@antoniamaia antoniamaia left a comment

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!!

Copy link
Member

@antoniamaia antoniamaia left a comment

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!! 🥇

Copy link
Member

@camilamaia camilamaia left a comment

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!!

@camilamaia camilamaia merged commit d7b645d into brazilian-utils:main Nov 18, 2024
8 checks passed
@jaimenunes jaimenunes deleted the 398 branch January 23, 2025 13:54
@camilamaia camilamaia mentioned this pull request Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Retornar Município e UF a partir do Código IBGE

3 participants