search

cuducos / minha-receita

🏢 Sua API web para consulta de informações do CNPJ da Receita Federal
https://minhareceita.org
MIT License
1.32k stars 132 forks source link

Corrige bug no comando `sample` adicionando o CSV de municípios #123

Closed cuducos closed 2 years ago

cuducos commented 2 years ago

Eu não posso testar manualmente agora. Se alguém confirmar que funciona, posso fazer o merge : )

cc @im-alexandre Fixes #121

im-alexandre commented 2 years ago

Testei aqui. Não funcionou. Ele tenta abrir como zip e fazer a amostra. Teria que copiar o arquivo mesmo. Vou alterar e mando um novo pull request

cuducos commented 2 years ago

Ele tenta abrir como zip e fazer a amostra.

Ah… verdade! Falha minha… obrigado por testar!

im-alexandre commented 2 years ago

Criei uma função de cópia e inclui na função Sample. Crio um pull request?

cuducos commented 2 years ago

Testei aqui no ce79434 adicionar uma função para lidar com amostra de CSV. Se quiser testar, senão eu olho no final do dia!

Criei uma função de cópia e inclui na função Sample. Crio um pull request?

Comentei no #122 que não sei o porquê fechasse o PR lá. Sempre pode. Eu abri esse PR pois achei que o seu estava fazendo muita coisa e resolvi tentar uma solução mais atômica — mas não sei se vai funcionar hahaha…

im-alexandre commented 2 years ago

Vou testar e aviso. Eu abri lá p poder mandar as alterações. Mas vou mandar aqui. Vou escrever um teste p isso e resolver só o lance do tabmun. Depois vamos vendo o resto.

cuducos commented 2 years ago

Depois vamos vendo o resto.

Uma das coisas do resto joguei na #124

cuducos commented 2 years ago

Converti para rascunho pois: