-
-
Notifications
You must be signed in to change notification settings - Fork 108
Create Municipality Method #412
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. 🚨 Try these New Features:
|
antoniamaia
left a comment
There was a problem hiding this 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!!
antoniamaia
left a comment
There was a problem hiding this 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!! 🥇
There was a problem hiding this 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!!
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
Checklist de Revisão
Comentários Adicionais (opcional)
Issue Relacionada
Closes #398