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] Concatenando campos de procedimentos clínicos #116

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

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

@vitmesquita vitmesquita changed the base branch from master to release October 15, 2024 20:14
@TanookiVerde TanookiVerde requested review from TanookiVerde and removed request for ThiagoTrabach October 18, 2024 17:26
@TanookiVerde TanookiVerde merged commit 0f71aa0 into release Oct 18, 2024
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