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

Removed python2 imports #4193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bskjois
Copy link

@bskjois bskjois commented Nov 1, 2024

Removed the python2 dependant modules as they are out of support now!

Signed-off-by: Kowshik Jois B S [email protected]

Removed the python2 dependant modules as they are out of support now!

Signed-off-by: Kowshik Jois B S <[email protected]>
@bskjois
Copy link
Author

bskjois commented Nov 6, 2024

@YongxueHong @PaulYuuu @MiriamDeng @maxujun
May I request to take a look and review this PR

Thanks in Advance!

@YongxueHong
Copy link
Contributor

Hi CC @peixiu
Could you help review this to see if it will affect your feature?
Thanks.

@PaulYuuu
Copy link
Contributor

I am okay with this change. Just considering the six module is also for 2 and 3 compatibility, do you have plan to remove it as well?

@peixiu
Copy link
Contributor

peixiu commented Nov 15, 2024

Hi CC @peixiu Could you help review this to see if it will affect your feature? Thanks.

Hi @heywji ,

Could you help to check if these change will impact virtio-input case running?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants