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.

Sign up for GitHub

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

Jump to bottom

lowercase schema for duckdb #102

Merged
merged 1 commit into from
Jul 11, 2024
Merged

lowercase schema for duckdb #102

merged 1 commit into from
Jul 11, 2024

Conversation

cnolanminich
Copy link
Contributor

This PR makes local dev for sling work again 🎉

@cnolanminich cnolanminich requested a review from izzye84 July 8, 2024 14:50
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

Your pull request at commit fe2b8d6032b12f75c2bfb147389e263f275b0655 is automatically being deployed to Dagster Cloud.

Location Status Link Updated
demo_assets View in Cloud Jul 08, 2024 at 03:04 PM (UTC)
data-eng-pipeline View in Cloud Jul 08, 2024 at 03:04 PM (UTC)
basics View in Cloud Jul 08, 2024 at 03:04 PM (UTC)
batch_enrichment View in Cloud Jul 08, 2024 at 03:04 PM (UTC)
snowflake_insights View in Cloud Jul 08, 2024 at 03:04 PM (UTC)

@izzye84
Copy link
Collaborator

izzye84 commented Jul 8, 2024

I tested it locally—good catch!

@cnolanminich cnolanminich merged commit 5e2e985 into master Jul 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@izzye84 izzye84 Awaiting requested review from izzye84

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cnolanminich @izzye84