docs: incorpora resultados dos testes do PNCP ComprasGOV - #71
docs: incorpora resultados dos testes do PNCP ComprasGOV#71LarrissaSilva wants to merge 1 commit into
Conversation
| O comando correto é `colibri lake query`, não `colibri lake q` — essa | ||
| abreviação aparece por engano em alguns exemplos mais antigos. | ||
| Use sempre o nome completo do comando, `colibri lake query`; abreviações que | ||
| aparecem em exemplos antigos não são aceitas pela CLI atual. |
There was a problem hiding this comment.
Talvez seja melhor fazer uma refatoração completa para usar a terminologia atual de forma consistente em toda a documentação?
| `baixar_catalogo.py`, `manifesto_bucket.py`, `salvar_arquivo_no_bucket.py`, | ||
| `csv_para_parquet.py`, `ducklake.py`, entre outros. | ||
|
|
||
| ## Validação local do PNCP ComprasGOV |
There was a problem hiding this comment.
Talvez fosse conveniente adicionar uma nova seção sobre Pipelines em geral e incluir os detalhes sobre o Comprasgov lá, e no futuro sobre outros Pipelines?
There was a problem hiding this comment.
Boa sugestão, @yolile! Concordo com essa organização. Vou aguardar também a revisão do Heitor e, depois, reúno os feedbacks para fazer os ajustes em um único commit.
There was a problem hiding this comment.
Também estava querendo escrever sobre cada pipeline, sua lógica, a origem dos dados, como ele é atualizado etc. Vamos fazer isso em uma nova seção, sim! :)
| exige rodar, na raiz do repositório: | ||
| exige rodar os testes automatizados, na raiz do repositório: |
There was a problem hiding this comment.
Acredito que, se meu PR de CI automático for combinado, esta seção e prática não serão mais necessárias. FYI
There was a problem hiding this comment.
Perfeito, @yolile! Obrigada pelo aviso. Quando o PR de CI for incorporado, posso revisar essa seção e retirar as instruções manuais que forem substituídas pela automação.
heitorgama
left a comment
heitorgama
left a comment
There was a problem hiding this comment.
Ótimos ajustes, Larrissa! Obrigado
| `baixar_catalogo.py`, `manifesto_bucket.py`, `salvar_arquivo_no_bucket.py`, | ||
| `csv_para_parquet.py`, `ducklake.py`, entre outros. | ||
|
|
||
| ## Validação local do PNCP ComprasGOV |
There was a problem hiding this comment.
Também estava querendo escrever sobre cada pipeline, sua lógica, a origem dos dados, como ele é atualizado etc. Vamos fazer isso em uma nova seção, sim! :)
|
Oi @LarrissaSilva! Complemento rápido, nada aqui parece precisar segurar o merge. Caçando comandos inexistentes hoje (mesma missão do seu PR), achei dois casos que já estavam na doc antes dele e seguem lá; deixo registrado para um follow-up:
De resto, as correções de exemplos (lake q, resumo_anual) batem certinho com o catálogo real de hoje. Aproveitando a passagem: o PR está aprovado e sem conflito, e pelo que li a única dúvida é de sequência. @LarrissaSilva, você ainda pretende aquele commit reunindo os feedbacks, ou as sugestões da @yolile podem virar follow-ups? (Uma delas, como a própria yolile apontou, depende do PR de CI #69 entrar primeiro.) Se a resposta for follow-up, @heitorgama, talvez já dê para mergear e a doc atualizada entra no ar |
|
@maia-andre, obrigada pela revisão e pelo levantamento. Como o comportamento do colibri lake years ainda vai ser decidido na #80 e uma das sugestões da @yolile depende do PR #69, acredito que seja melhor tratar esses pontos como follow-ups, incluindo a correção de colibri lake tabelas para colibri lake tables. Da minha parte, não pretendo adicionar outro commit neste PR. @heitorgama, pode fazer o merge quando considerar adequado. Obrigada. |
4 participants
O que muda com este PR
Este PR integra à documentação os resultados dos testes locais realizados no projeto Colibri, principalmente no pipeline PNCP ComprasGOV.
As alterações:
--bucketem ambientes de teste;Como testar
git diff --check.dbt/modelos/marts/pncp_comprasgov.resumo_anual,lake q,colibri doctorou opções ainda não implementadas.@heitorgama e @yolile, poderiam avaliar esta atualização da documentação, por favor?