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

misc: Update python versions used for pyvelox #12157

Closed

Conversation

czentgr
Copy link
Collaborator

@czentgr czentgr commented Jan 23, 2025

The current minimum is python 3.7. but that has passed EOL. This change sets the minimum at the oldest supported version 3.9.

@czentgr czentgr requested review from kgpai and assignUser January 23, 2025 20:49
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 23, 2025
Copy link

netlify bot commented Jan 23, 2025

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 1d41b88
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/6793e24ccd491300095fba04

Copy link
Collaborator

@assignUser assignUser left a comment

Choose a reason for hiding this comment

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

Can you touch something in velox/ to trigger the fuzzer jobs (which are using pyvelox)

The current minimum is python 3.7. but that has passed EOL.
This change sets the minimum at the oldest supported version 3.9.
@czentgr czentgr force-pushed the cz_update_python_in_setup branch from 481c35b to ec18a87 Compare January 24, 2025 18:53
Ensure fuzzer jobs run when the setup.py is updated.
@czentgr
Copy link
Collaborator Author

czentgr commented Jan 24, 2025

@assignUser I made another commit to run the fuzzer jobs on setup.py change.

@czentgr czentgr marked this pull request as ready for review January 27, 2025 20:10
@czentgr czentgr requested a review from majetideepak as a code owner January 27, 2025 20:10
@czentgr
Copy link
Collaborator Author

czentgr commented Jan 29, 2025

@assignUser Please review. You think this is ok with the two commits?

@assignUser assignUser added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Jan 29, 2025
@facebook-github-bot
Copy link
Contributor

@bikramSingh91 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@bikramSingh91 merged this pull request in 256ed01.

ArnavBalyan pushed a commit to ArnavBalyan/velox that referenced this pull request Jan 31, 2025
Summary:
The current minimum is python 3.7. but that has passed EOL. This change sets the minimum at the oldest supported version 3.9.

Pull Request resolved: facebookincubator#12157

Reviewed By: kgpai

Differential Revision: D68863807

Pulled By: bikramSingh91

fbshipit-source-id: 46cc43b4b418b3eb8c3681cd7548e5f4a3f68f13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants