-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,6 +16,8 @@ Contributions in the form of issues and pull requests are very much welcome here | |
- We default to building a view and consider switching to a table or incremental table if performance becomes an issue. | ||
|
||
## [BETA] Pre-push hooks | ||
UPDATE: These pre-push hooks require running `dbt compile` which is a fairly slow step due to the size of our project. We intend to rewrite these hooks to be more efficient but for the time being they remain cumbersome. Feel free to use them if you find them useful but the same checks will run in a Github Action when you commit your code. Feel free to uninstall if they do not bring joy, we'll let wizards know when we think we've improved them enought to warrant making them part of the general development flow. | ||
This comment has been minimized.
Sorry, something went wrong.
This comment has been minimized.
Sorry, something went wrong. |
||
|
||
We are testing out adding pre-push hooks to our workflow. The goal is to catch common errors before code is pushed and | ||
streamline the pull request review process. | ||
|
||
|
@@ -152,4 +154,4 @@ Example from dex_trades.sql | |
"dex", | ||
\'["jeff-dude", "hosuke", "0xRob"]\') }}' | ||
) | ||
``` | ||
This comment has been minimized.
Sorry, something went wrong.
dekefg
|
||
``` | ||
This comment has been minimized.
Sorry, something went wrong.
dekefg
|
0x00409fC839a2Ec2e6d12305423d37Cd011279C09