-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
Conversão de código do estado do IBGE para a sigla do Estado #410
Conversation
- Adicionado função responsável por converter o código de estado do IBGE na sigla do estado - Adicionado testes unitários
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #410 +/- ##
==========================================
+ Coverage 99.77% 99.78% +0.01%
==========================================
Files 17 18 +1
Lines 437 472 +35
==========================================
+ Hits 436 471 +35
Misses 1 1 ☔ View full report in Codecov by Sentry. |
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.
Oii @carlos-moreno, faltaram dois detalhes importantes: o item 10. Atualizar READMEs e o item 12. Adicione Entradas no CHANGELOG.md.. Muito obrigada pela contribuição!! 🎉
Co-authored-by: Fernando Bezerra [email protected] |
README.md
Outdated
* code (str): O código IBGE de 2 dígitos a ser convertido. | ||
|
||
Retorna: | ||
* str or None: O código UF correspondente ao código IBGE, ou Nenhum se 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.
Dá para deixar o tipo em inglês mesmo, porque assim fica claro do que se está falando. Só para não confundir mesmo.
* str or None: O código UF correspondente ao código IBGE, ou Nenhum se o | |
* str or None: O código UF correspondente ao código IBGE, ou None se o |
tests/ibge/test_uf.py
Outdated
self.assertIsNone(convert_code_to_uf("")) # Código vazio | ||
self.assertIsNone(convert_code_to_uf("AB")) # Código não numérico | ||
|
||
# implementar mais casos de teste aqui se necessário |
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.
Como vcs não sentiram a necessidade de implementar mais testes, dá só para arrancar fora esse comentário mesmo. Ele era só para ajudar a guiar.
# implementar mais casos de teste aqui se necessário |
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, ficou lindão!! 👏
Só comentei dois pequenos detalhes, mas fora isso já tá pronto para mergear. Se vcs quiserem, eu mesma posso atualizar aqui com as modificações e já dale.
…ções do code review da Camila Maia.
Boa, valeu pelas dicas @camilamaia |
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.
Mandaram ver!! Obrigada pela contribuição e participação no tutorial, pessoal!! 🌟
Co-authored-by: Fernando Bezerra <[email protected]>
Descrição
Converter o código de estado do IBGE na sigla do Estado
Mudanças Propostas
Adicionado função responsável por realizar a conversão do código de estado do IBGE na sigla do estado.
convert_code_to_uf
.Checklist de Revisão
Comentários Adicionais (opcional)
Issue Relacionada
Closes #397