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

versatile-data-kit: Support for Py3.12 #3143

Merged
merged 9 commits into from
Feb 26, 2024
Merged

Conversation

gabrielgeorgiev1
Copy link
Contributor

@gabrielgeorgiev1 gabrielgeorgiev1 commented Feb 21, 2024

This PR adds a CICD job checking that components and plugins can be built and tested in a Py3.12 environment.
vdk-singer does not support 3.12 as its dependency tap-rest-api-msdk explicitly does not support it.
vdk-ingest-http's tests seem to fail in an odd way - this is described in this issue I've created - #3153

Testing done: cicd

This PR adds a CICD job checking that components and plugins
can be built and tested in a Py3.12 environment.

Testing done: cicd

Signed-off-by: Gabriel Georgiev <[email protected]>
@antoniivanov
Copy link
Collaborator

tap-rest-api-msdk is not a dependency of vdk-singer. It is a dependency of its tests. But we can use another tap for testing.

Also vdk-singer is a pre-alpha plugin (aka prototype) . So let's not put too much effort for now there. We can just open a ticket and address it when we decide to invest more in vdk-singer.

Copy link
Collaborator

@antoniivanov antoniivanov left a comment

Choose a reason for hiding this comment

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

Looks good to me. For those plugins that have 3.12 tests we do not need 3.11 tests.

I think only vdk-core is tested with every version (as it is the core component :) )

Copy link
Contributor

@yonitoo yonitoo left a comment

Choose a reason for hiding this comment

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

LGTM! Only the removal of 3.11 tests remains, maybe it could be done as part of another PR as well.

Signed-off-by: Gabriel Georgiev <[email protected]>
Signed-off-by: Gabriel Georgiev <[email protected]>
@gabrielgeorgiev1 gabrielgeorgiev1 merged commit d90c97c into main Feb 26, 2024
8 checks passed
@gabrielgeorgiev1 gabrielgeorgiev1 deleted the person/gageorgiev/py312 branch February 26, 2024 13:55
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.

4 participants