-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
fixed comment util.py #8222
fixed comment util.py #8222
Conversation
Codecov Report
@@ Coverage Diff @@
## main #8222 +/- ##
==========================================
- Coverage 86.23% 86.22% -0.01%
==========================================
Files 174 174
Lines 25518 25518
==========================================
- Hits 22005 22004 -1
- Misses 3513 3514 +1
|
@d-kaneshiro can you please rebase this off main? We made some CI improvements to split up our integration tests to run faster. Unfortunately it means you'll need to pull main in to get all your checks passing. |
@emmyoop |
@d-kaneshiro I'm looking into what's happening here. This is something we will probably need to fix. |
@d-kaneshiro if you could rebase off main one more time CI should run and pass! |
1adfe15
to
fc11bc5
Compare
Thank you. |
resolves # None
Issue is not created because it corresponds to the following cases.
https://docs.getdbt.com/community/resources/oss-expectations
Problem
There is a possibility of executing the wrong command when using the dbt command in pytest.
Solution
Fixed the comment to run correctly.
Checklist