-
Notifications
You must be signed in to change notification settings - Fork 399
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] reduce spurious diffs to snowflake_view statement #171
Conversation
Go modules have gotten good enough that vendoring does not seem worth the cost any more. This removes the vendor directory and adds a step to our build to detect some problems with go.{mod,sum} being out of date.
Make an effort to normalize queries that create views. Snowflake does not preserve formatting when round-tripping these queries, so to compare changes we need to do some normalization. See the comment in the code for limitations.
Codecov Report
@@ Coverage Diff @@
## master #171 +/- ##
==========================================
- Coverage 67.87% 67.86% -0.02%
==========================================
Files 46 46
Lines 3088 3087 -1
==========================================
- Hits 2096 2095 -1
Misses 717 717
Partials 275 275
Continue to review full report at Codecov.
|
Co-Authored-By: Eduardo Lopez <elopez@chanzuckerberg.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.
this is great, thank you
Make an effort to normalize queries that create views.
Snowflake does not preserve formatting when round-tripping these
queries, so to compare changes we need to do some normalization. See the
comment in the code for limitations.
Test Plan
References