Skip to content
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

More tests for custom grain #1664

Open
wants to merge 4 commits into
base: court/custom-offset10
Choose a base branch
from

Conversation

courtneyholcomb
Copy link
Contributor

I had some random follow up tasks to add tests for specific scenarios, and some simple ones are covered here.

@courtneyholcomb courtneyholcomb added Skip Changelog Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment labels Feb 7, 2025
@courtneyholcomb courtneyholcomb requested a review from a team as a code owner February 7, 2025 00:06
@cla-bot cla-bot bot added the cla:yes label Feb 7, 2025
@courtneyholcomb courtneyholcomb added Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment and removed Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment labels Feb 7, 2025
@courtneyholcomb courtneyholcomb deployed to DW_INTEGRATION_TESTS February 7, 2025 00:06 — with GitHub Actions Active
@github-actions github-actions bot removed the Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment label Feb 7, 2025
@@ -881,3 +881,12 @@ metric:
offset_window: 1 alien_day
alias: bookings_offset
- name: bookings
---
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small nit to remove extra +---

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants