-
-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
gogonzo
commented
Jun 29, 2023
•
edited
Loading
edited
- teal Decouple scda teal#858
- teal.widgets
- osprey Decouple scda osprey#121
- teal.osprey Decouple scda teal.osprey#214
- teal.transform Decouple scda teal.transform#139
@@ -27,12 +24,6 @@ downstream_repos: | |||
insightsengineering/teal.modules.general: |
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.
Downward dependency
As of this insightsengineering/teal.transform#133 scda is going to be decoupled from teal.trasform once merged . this should be removed
@@ -15,24 +15,12 @@ downstream_repos: | |||
insightsengineering/goshawk: | |||
repo: insightsengineering/goshawk | |||
host: https://github.com | |||
insightsengineering/teal: | |||
repo: insightsengineering/teal | |||
host: https://github.com | |||
insightsengineering/teal.data: | |||
repo: insightsengineering/teal.data | |||
host: https://github.com |
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.
There is a task assigned for insightsengineering/teal.data#145, which involves removing scda from teal.data. I'm a little uncertain about whether this removal should be done now or at a later time. Should pull request be kept it on hold until the task is merged?
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.
We can merge what is ready now and open decouple_scda@main
again for other PRs. It's not a problem.
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.
approved
closes insightsengineering/teal.osprey#213 see also: - insightsengineering/scda#125 - insightsengineering/scda.2022#123 --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
closes #213 #204 see also: - insightsengineering/scda#125 - insightsengineering/scda.2022#123
closes #834 see also: - insightsengineering/scda#125 - insightsengineering/scda.2022#123 --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: 27856297+dependabot-preview[bot]@users.noreply.github.com <27856297+dependabot-preview[bot]@users.noreply.github.com>