-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Source Amplitude: unpin CDK #35987
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
Source Amplitude: unpin CDK #35987
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
@@ -17,7 +17,7 @@ include = "source_amplitude" | |||
|
|||
[tool.poetry.dependencies] | |||
python = "^3.9,<3.12" | |||
airbyte-cdk = "==0.52.0" | |||
airbyte-cdk = "^0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's way better to have >=0.52.0
, but not blocking the update overall.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the review! I think we're specifically using the ^0
notation to prevent automatic major version updates, but I'll double check with Natik to confirm before unpinning the rest of the connectors 👍
What
Unpinning the CDK for source Amplitude.
How