Conversão de código do estado do IBGE para a sigla do Estado#410
Conversão de código do estado do IBGE para a sigla do Estado#410camilamaia merged 5 commits intobrazilian-utils:mainfrom
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. |
antoniamaia
left a comment
antoniamaia
left a comment
There was a problem hiding this comment.
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.
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.
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 |
camilamaia
left a comment
camilamaia
left a comment
There was a problem hiding this comment.
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 |
antoniamaia
left a comment
antoniamaia
left a comment
There was a problem hiding this comment.
Mandaram ver!! Obrigada pela contribuição e participação no tutorial, pessoal!! 🌟
Co-authored-by: Fernando Bezerra <[email protected]>
3 participants
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