You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copied from #5088, which we are closing in favor of this because we're not actually seeing the test failure now.
Having a whole separate file for a global selected_resources variable in core/dbt/selected_resource.py is kind of overkill. It should be in some other file. In addition we should try hard to implement it in a way that doesn't use a global file variable, because they are a bad idea in general (despite the fact that we have a number of them) and they all need to be reset for tests to run properly (and quite possibly any persistent server code).
The text was updated successfully, but these errors were encountered:
github-actionsbot
changed the title
Make selected_resource non-global
[CT-1299] Make selected_resource non-global
Oct 5, 2022
This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please comment on the issue or else it will be closed in 7 days.
Although we are closing this issue as stale, it's not gone forever. Issues can be reopened if there is renewed community interest. Just add a comment to notify the maintainers.
Copied from #5088, which we are closing in favor of this because we're not actually seeing the test failure now.
Having a whole separate file for a global selected_resources variable in core/dbt/selected_resource.py is kind of overkill. It should be in some other file. In addition we should try hard to implement it in a way that doesn't use a global file variable, because they are a bad idea in general (despite the fact that we have a number of them) and they all need to be reset for tests to run properly (and quite possibly any persistent server code).
The text was updated successfully, but these errors were encountered: