feat:retorna codigo do ibge do nome do municipio e da uf #411
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #411 +/- ##
=======================================
Coverage 99.78% 99.79%
=======================================
Files 18 19 +1
Lines 472 495 +23
=======================================
+ Hits 471 494 +23
Misses 1 1 ☔ View full report in Codecov by Sentry. |
|
@camilamaia obrigado pela ajuda, em breve aplicarei as mudanças! |
|
|
||
| Retorna: | ||
| * str: O código IBGE do município. Retorna None se o nome não for válido ou não existir. | ||
| Exemplo: |
There was a problem hiding this comment.
pular linha aqui por conta da formatação
| * str: O código IBGE do município. Retorna None se o nome não for válido ou não existir. | ||
| Exemplo: | ||
|
|
||
| ``` |
| @@ -1110,6 +1112,41 @@ Exemplo: | |||
| 'RJ' | |||
| >>> convert_code_to_uf("99") | |||
| >>> | |||
| >>>>>>> main | |||
There was a problem hiding this comment.
essa modificação está correta?
There was a problem hiding this comment.
lembra os passos/comandos para gerar esse arquivo?
seria interessante ter essa referência para uma possível automação futura
There was a problem hiding this comment.
Boa pergunta @mateusoliveira43!
@CarduCaldeira consegue passar para gente como foi gerado?
| return code | ||
|
|
||
|
|
||
| def _transform_text(municipality_name: str): # type: (str) -> str |
There was a problem hiding this comment.
valeria escrever teste só para essa função?
There was a problem hiding this comment.
Como é uma função privada e parece que eles já estão testando o seu comportamento na função pública, acredito que seja meio que opcional. Você concorda @mateusoliveira43
|
|
||
| Retorna: | ||
| * str: O código IBGE do município. Retorna None se o nome não for válido ou não existir. | ||
| Exemplo: |
There was a problem hiding this comment.
Boa @mateusoliveira43 ! Vou adicionar a sugestão aqui:
| Exemplo: | |
| Exemplo: |
| * str: O código IBGE do município. Retorna None se o nome não for válido ou não existir. | ||
| Exemplo: | ||
|
|
||
| ``` |
There was a problem hiding this comment.
| ``` | |
| ```python |
There was a problem hiding this comment.
Boa pergunta @mateusoliveira43!
@CarduCaldeira consegue passar para gente como foi gerado?
| return code | ||
|
|
||
|
|
||
| def _transform_text(municipality_name: str): # type: (str) -> str |
There was a problem hiding this comment.
Como é uma função privada e parece que eles já estão testando o seu comportamento na função pública, acredito que seja meio que opcional. Você concorda @mateusoliveira43
Descrição
Implementa uma função que recebe o nome do municipio e a sigla da UF e retorna seu código do IBGE.
@mateusoliveira43, @pedromendesjr, @nettorobson
Mudanças Propostas
Checklist de Revisão
Comentários Adicionais (opcional)
Estamos na dúvida na utilização de um arquivo json para leitura dos códigos referentes ao municípios (caso implementado) ou utilização de uma API externa para obter tais códigos.
Uma sugestão de desenvolvimento para melhoria seria a criação de um processo de automação para atualização do json
Issue Relacionada
Closes #399