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

Hci/review/thiago #81

Merged
merged 113 commits into from
Aug 28, 2024
Merged

Hci/review/thiago #81

merged 113 commits into from
Aug 28, 2024

Conversation

ThiagoTrabach
Copy link
Contributor

@ThiagoTrabach ThiagoTrabach commented Aug 28, 2024

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).

d116626 and others added 30 commits July 29, 2024 17:05
TanookiVerde and others added 28 commits August 26, 2024 11:31
…sql and int_historico_clinico__paciente__vitai.sql
…ciente__vitai and int_historico_clinico__paciente__vitacare
…ergia.sql and _mart_historico_clinico_schema.yml
….sql, mart_historico_clinico__medicamento_cronico.sql, and mart_historico_clinico__paciente.sql
@ThiagoTrabach ThiagoTrabach merged commit 6b209d4 into master Aug 28, 2024
3 of 7 checks passed
@ThiagoTrabach ThiagoTrabach deleted the hci/review/thiago branch August 28, 2024 18:35
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.

4 participants