cuducos / minha-receita

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

Fazer com que o comando `sample` copie o arquivo `updated_at.txt` caso ele exista #152

Closed cuducos closed 1 year ago

eu-ovictor commented 1 year ago

Gostaria de contribuir nessa issue, poderia me dar um pouco mais de contexto sobre ela por favor?

cuducos commented 1 year ago

Só para saber por onde começar… o que você entende e o que você não entende dessa issue?

Alguns links:

O caso de uso seria mais ou menos o seguinte:

  1. baixei todos os dados e quero testar o ETL (ou seja, $ minha-receita transform) mas não quero carregar todos os dados (por exemplo, quero testar se o processo funciona, e não necessariamente criar um banco de dados completo)
  2. para isso, antes o comando sample
  3. Se faço isso, e executo o transform -d diretório-que-usei-no-sample vai dar erro pois não existe o updated_at.txt no diretório-que-usei-no-sample

É esse erro que o PR que fecha essa issue deve evitar.

eu-ovictor commented 1 year ago

No caso de não existir o arquivo updated_at.txt no diretório de origem do sample a app deveria criar um com o valor mockado para ser usado no transform? Ou deve apenas ignorar?

cuducos commented 1 year ago

Bela perguinta! Não tinha pesando nisso…

Pensando alto aqui:

O que acha?

eu-ovictor commented 1 year ago

Acho uma boa. Para a opção --mock-updated-at a gente poderia permitir a pessoa passar uma data como argumento, caso ela não passe, podemos usar a data do dia atual como valor default, no formato aaaa-mm-dd que é como ele criado no download. O que acha?

cuducos commented 1 year ago

Perfeito : )