Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: uncomment sistema_origem filter in estoque position model #230

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

ThiagoTrabach
Copy link
Contributor

Descrição & motivação

Issues Relacionadas

Tipo de Mudança

  • Novo modelo
  • Correção de bug
  • Refatoração
  • Mudança que quebra compatibilidade
  • Documentação
  • Atualização/Instalação de dependências

A fazer antes do merge

Em caso de quebra de compatibilidade

  • Validar em dev o funcionamento correto dos reports, dashboard e modelos afetados

DAG de Linhagem:

Validação dos modelos:

Considerações de Impacto:

Mudanças nos modelos existentes:

Lista de Verificação:

  • Meu pull request representa uma peça lógica de trabalho.
  • Meus commits estão relacionados ao pull request e parecem limpos.
  • Meu SQL segue o guia de estilo da SMS-Rio.
  • dbt build completa com sucesso e os testes dbt passam (se não, detalhe por quê)
  • Materializei meus modelos apropriadamente (tipo de materialização e no dataset correto).
  • Adicionei testes e documentação apropriados a quaisquer modelos novos.
  • Adicionei policy tags apropriadas a quaisquer modelos novos.
  • Adicionei tags apropriadas a quaisquer modelos novos.
  • Atualizei o arquivo README (quando necessário).

Copy link

Modelos Modificados

Os seguintes modelos foram modificados:
- models/marts/dit/estoque/mart_estoque__posicao_atual.sql

Comando para rodar DBT

dbt run --select mart_estoque__posicao_atual+ --full-refresh

@TanookiVerde TanookiVerde merged commit cfc3e12 into master Feb 20, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants