Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove warning: the except macro is now provided in dbt Core #94

Merged
merged 1 commit into from
Oct 20, 2022
Merged

Remove warning: the except macro is now provided in dbt Core #94

merged 1 commit into from
Oct 20, 2022

Conversation

wojtek-szymanski
Copy link
Contributor

Removing dbt waring:

Warning: the `except` macro is now provided in dbt Core. It is no longer available in dbt_utils and backwards compatibility will be removed in a future version of the package. Use `except` (no prefix) instead. The ga_snowflake.test_my_first_dbt_model model triggered this warning.

@cdiniz cdiniz added the safe to test Label used to identify prs safe to test on the integration environments label Oct 20, 2022
@cdiniz cdiniz temporarily deployed to test October 20, 2022 14:34 Inactive
@cdiniz cdiniz temporarily deployed to test October 20, 2022 14:34 Inactive
@cdiniz cdiniz temporarily deployed to test October 20, 2022 14:34 Inactive
@cdiniz cdiniz temporarily deployed to test October 20, 2022 14:34 Inactive
@cdiniz cdiniz temporarily deployed to test October 20, 2022 14:42 Inactive
@github-actions github-actions bot removed the safe to test Label used to identify prs safe to test on the integration environments label Oct 20, 2022
cdiniz
cdiniz approved these changes Oct 20, 2022
View reviewed changes
Copy link
Collaborator

@cdiniz cdiniz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @wojtek-szymanski

@cdiniz cdiniz merged commit 5b2a92f into EqualExperts:master Oct 20, 2022
@cdiniz cdiniz mentioned this pull request Oct 20, 2022
Closed
@alaneve alaneve mentioned this pull request Mar 15, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cdiniz cdiniz cdiniz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wojtek-szymanski @cdiniz