-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
automate docker release #9833
automate docker release #9833
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9833 +/- ##
==========================================
- Coverage 88.12% 88.07% -0.05%
==========================================
Files 178 178
Lines 22458 22458
==========================================
- Hits 19790 19779 -11
- Misses 2668 2679 +11
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
c96ba5d
to
38e0eb1
Compare
e0c6456
to
df75bcc
Compare
8bab51b
to
df75bcc
Compare
.github/workflows/release-docker.yml
Outdated
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.
This file was moved to dbt-release
in dbt-labs/dbt-release#108 since it's shared by all adapters + core
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.
Conditional approval, assuming the defaults that looks like testing parameters are removed prior to merging.
resolves dbt-labs/dbt-release#109
Problem
We miss releasing docker images often
Solution
Delete Docker release workflow and move it to dbt-release since it's not just used by the core team dbt-labs/dbt-release#108
Add docker generation to end of release
Checklist