-
Notifications
You must be signed in to change notification settings - Fork 159
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 default group_id to init params #399
Conversation
👷 Deploy Preview for amazing-pothos-a3bca0 processing.
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #399 +/- ##
=======================================
Coverage 79.36% 79.36%
=======================================
Files 48 48
Lines 1691 1691
=======================================
Hits 1342 1342
Misses 349 349
☔ View full report in Codecov by Sentry. |
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.
Thanks a lot for the contribution, @chrishronek !
At the moment some tests are failing, could you have a look into them?
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 looks good! integration tests are failing bc external contributes don't have access to GH secrets, but we're working to fix it
## Description When attempting to override the `group_id` parameter of a DbtTaskGroup, I get the following error: ```text TypeError: __init__() got multiple values for argument 'group_id' ``` ## Related Issue(s) closes #398 ## Breaking Change? There are no breaking changes with this PR ## Checklist - [x] I have made corresponding changes to the documentation (if required) - [x] I have added tests that prove my fix is effective or that my feature works Co-authored-by: Julian LaNeve <[email protected]>
Description
When attempting to override the
group_id
parameter of a DbtTaskGroup, I get the following error:Related Issue(s)
closes #398
Breaking Change?
There are no breaking changes with this PR
Checklist