-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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-paypal-transaction: ensure inline schemas, updated cdk, poetry (where possible) #36654
source-paypal-transaction: ensure inline schemas, updated cdk, poetry (where possible) #36654
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
LGTM. Connector version increment check freaks out, but this is fine. Good to merge in the morning.
Seems likely there were other updates on the master branch. Have to mess around with the version probably. |
@bleonard I was able to bump the CDK version last week for the |
…-paypal-transaction
rebased! |
24b1f77
to
e8cf1da
Compare
@@ -17,7 +17,7 @@ include = "source_paypal_transaction" | |||
|
|||
[tool.poetry.dependencies] | |||
python = "^3.9,<3.12" | |||
airbyte-cdk = "^0" | |||
airbyte-cdk = "0.82.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.
Why would we pin the versions again?
…-paypal-transaction
…-paypal-transaction
This was created from a set of automated scripts. In each case, not every update was needed for every connector, but overall here is what happened:
auto-schema update -s all description
: added descriptions to json and inline schemasconnector-code migrate_to_yaml -c all --type source
: migrates json schemas to connectors with a manifestairbyte-ci connectors --name=<all modified connectors from above> migrate_to_base_image
: makes sure that each is using a base docker image and updates docsairbyte-ci connectors --name=<all modified connectors from above> migrate-to-poetry
: moves connectors not already using poetry to do so and updates documentationairbyte-ci connectors --name=<all modified connectors from above> up_to_date
: updated the CDK to newer (0.80.0) versionThe version number and changelogs were also bumped using the
connector-code pulls
command.