-
Notifications
You must be signed in to change notification settings - Fork 124
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
Add postgres support #368
Add postgres support #368
Conversation
Passing postgres integration tests - https://github.com/dpguthrie/dbt_artifacts/actions/runs/5538393942 |
Hi @dpguthrie . Thanks for taking the time to add this functionality in. One of the team will spend some time reviewing it and get back to you. |
Would love to see this in the next release! |
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.
Thank you for your work on this. Sorry it's taken a while to review - there have been a number of changes to improve the backend, so I wanted to make sure those had been made first.
Overview
Update type - breaking / non-breaking
What does this solve?
Support for a new platform, Postgres
Outstanding questions
What databases have you tested with?