-
Notifications
You must be signed in to change notification settings - Fork 497
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
Fix: "except" field in Star macro handles cross db column casing #397
Fix: "except" field in Star macro handles cross db column casing #397
Conversation
3da448c
to
aaac6a3
Compare
@jtcohen6 failing integration tests appear to be unrelated to this PR. Tried pinning dbt_requirements in integration_tests project even though it inherits from root directory but didn't help. I think it's occurring in Anyways hoping this PR can make it through. Let me know if any additional information is required. Error encountered in /home/********/project/integration_tests/dbt_modules/dbt_utils
Running with dbt=0.21.0-b1
Encountered an error:
Runtime Error
Failed to read package: Runtime Error
This version of dbt is not supported with the 'dbt_utils' package.
Installed version of dbt: =0.21.0-b1
Required version of dbt for 'dbt_utils': ['>=0.20.0', '<0.21.0']
Check the requirements for the 'dbt_utils' package, or run dbt again with --no-version-check |
@danieldiamond This is related to how we run |
#403 If one of our PR's gets merged, we should close the other @danieldiamond |
niiice @reidwil-jb didnt realize that |
@danieldiamond yep! Two solutions to solve the same problem! Would love to have this merged asap as the capitalization has tripped me up multiple times 😄 |
Hi @danieldiamond - thank you so much for contributing to dbt-utils! Given the duplicate PRs, we are going to close this and merge #403. |
This is a:
master
dev/
branchdev/
branchDescription & motivation
Checklist
star()
source)