Adicionei municipio Boa Esperança do Norte - MT ao arquivo MunIBGE-UF51#397
Conversation
|
Olá! Estou começando com contribuições open source e utilizei o informe técnico oficial para inserir o município corretamente. Fico à disposição para ajustar o que for necessário. Obrigado pela atenção! |
|
Olá! Fiz a correção da tabulação conforme a estrutura do arquivo. Notei que os testes automáticos estão falhando, mas acredito que não se trata de um problema com minha alteração. Fico à disposição para qualquer ajuste. Obrigado! |
leandroguimaraes
left a comment
leandroguimaraes
left a comment
There was a problem hiding this comment.
Olá @FishmmansCavalcante ,
Além dos comentários que adicionei, acrescento que os testes falhando indicam que os unit tests em tests/test_utils.py precisam ser atualizados. Verifique as mensagens de erro do pipeline para mais detalhes sobre isso.
| @@ -1,4 +1,4 @@ | |||
| 5100102 Acorizal | |||
| 5100102 Acorizal | |||
There was a problem hiding this comment.
Essa alteração está fora do escopo do seu PR. Sugiro que seja desfeita.
There was a problem hiding this comment.
@FishmmansCavalcante essa alteração não foi desfeita.
pynfe/data/MunIBGE/MunIBGE-UF51.txt
Outdated
| 5101704 Barra do Bugres | ||
| 5101803 Barra do Garças | ||
| 5101852 Bom Jesus do Araguaia | ||
| 5101852 Bom Jesus do Araguaia |
There was a problem hiding this comment.
Essa alteração está fora do escopo do seu PR. Sugiro que seja desfeita.
| 5108956 Nova Monte Verde | ||
| 5108956 Nova Monte Verde | ||
|
|
There was a problem hiding this comment.
Essa alteração está fora do escopo do seu PR. Sugiro que seja desfeita.
pynfe/data/MunIBGE/MunIBGE-UF51.txt
Outdated
| 5101803 Barra do Garças | ||
| 5101852 Bom Jesus do Araguaia | ||
| 5101852 Bom Jesus do Araguaia | ||
| 5108404 Boa Esperança do Norte |
There was a problem hiding this comment.
Segundo o Informe Técnico 2025.001 - v.1.00 - Publicado em 18/02/2025, e o site do IBGE, o código correto do município de Boa Esperança do Norte é o 5101837.
There was a problem hiding this comment.
Acrescento que, para manter a ordem alfabética do arquivo, Boa Esperança do Norte deveria vir antes de Bom Jesus do Araguaia.
|
Closes #386. |
| 5108857 Nova Marilândia | ||
| 5108907 Nova Maringá | ||
| 5108956 Nova Monte Verde | ||
|
|
There was a problem hiding this comment.
@FishmmansCavalcante essa modificação não deveria fazer parte do seu PR.
| def test_normalizar_municipio_boa_esperanca_do_norte(self): | ||
| self.assertEqual(normalizar_municipio("Boa Esperança do Norte"),"BOA ESPERANCA DO NORTE") | ||
|
|
There was a problem hiding this comment.
Aqui você está testando a normalização do "ç", mas esse teste não é necessário, pois isso já está coberto por este teste: https://github.com/TadaSoftware/PyNFe/blob/main/tests/test_utils.py#L62
|
Corrigido e adicionado neste PR |
4 participants
Este pull request adiciona o município de Boa Esperança do Norte - MT ao arquivo
MunIBGE-UF51.txt, conforme publicado no Informe Técnico 2025.001 v.1.00 do ENCAT.5108404