Catch staging up to main #22
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…or BQ (#18) Testing CI jobs, we want both BQ and SF to run here. --------- Co-authored-by: Winnie Winship <winnie@fishtownanalytics.com> Co-authored-by: dave-connors-3 <73915542+dave-connors-3@users.noreply.github.com>
There is a race condition using multiple warehouses for CI jobs on the same repo. This adds GitHub Actions to run the CI jobs so we can check against all of them. It also adds slim CD jobs that run on merge to main.
Closes #1 This turned into pulling a sweater thread that unravelled some issues in the source data, so I had to dig in to quite a bit! This updates some issues in the source data seed files in addition to fixing logic.
The Semantic Layer has an awesome feature to allow the construction and materialization of queries based on your metrics. We've added those to the Jaffle Shop and expanded the SL implementation a bit.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We still haven't implemented the full W-A-P flow yet so
stagingis behindmain, fixing that and then we'll transition to the proper flow.