dagster
dagster copied to clipboard
dagster-io
Unpin google dependencies
Summary & Motivation
Fixes https://github.com/dagster-io/dagster/issues/4748
How I Tested These Changes
I installed dagster in my local env (thanks for the Contributing guide!) and ran
python -m pytest python_modules/libraries/dagster-gcp/dagster_gcp_tests
The only errors i got were access error because i couldn't read the gcs bucket.
Let me know if something else needs to be tested! (I don't use dataproc though so that would be the hardest to test for me)
@amarrella is attempting to deploy a commit to the Elementl Team on Vercel.
A member of the Team first needs to authorize it.
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
| Name | Status | Preview | Comments | Updated |
|---|---|---|---|---|
| dagit-storybook | ⬜️ Ignored (Inspect) | Oct 10, 2022 at 8:33PM (UTC) | ||
| dagster | ⬜️ Ignored (Inspect) | Oct 10, 2022 at 8:33PM (UTC) | ||
| dagster-oss-cloud-consolidated | ⬜️ Ignored (Inspect) | Oct 10, 2022 at 8:33PM (UTC) |
@rexledesma saw that you added the original diff that pinned google-api-python-client, mind taking a look at this?
Hi @clairelin135 @rexledesma is there anything I can do to help this PR getting merged? These pins are holding us back on several dependencies
@amarrella sorry about the late response! This is looking good to me, I've gone ahead and merged this now