-
Notifications
You must be signed in to change notification settings - Fork 58
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
Addition of medical econ tables for Paul OA #692
base: main
Are you sure you want to change the base?
Conversation
@@ -3,7 +3,7 @@ version: '0.13.0' | |||
config-version: 2 | |||
require-dbt-version: ">=1.5.0" | |||
|
|||
profile: default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kuljisd can you remove your changes to this file? Local testing config shouldn't be pushed in a branch.
@@ -2,7 +2,7 @@ name: 'integration_tests' | |||
version: '1.0.0' | |||
config-version: 2 | |||
|
|||
profile: default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kuljisd same issue with this one. Local testing config shouldn't be pushed in a branch.
Describe your changes
Addition of medical_economics schema. Within said folder power bi data tables will be created for data visualization.
How has this been tested?
Please describe the tests you ran to verify your changes. Provide instructions or code to reproduce output.
Reviewer focus
Please summarize the specific items you’d like the reviewer(s) to look into.
Checklist before requesting a review
tuva_last_run
to the final outputPackage release checklist
dbt_project.yml
(Optional) Gif of how this PR makes you feel
Loom link