Skip to content

Upgrade type hint and others to Python 3.9#8814

Merged
NicolasHug merged 3 commits intopytorch:mainfrom
cyyever:ruff39
Apr 23, 2025
Merged

Upgrade type hint and others to Python 3.9#8814
NicolasHug merged 3 commits intopytorch:mainfrom
cyyever:ruff39

Conversation

@cyyever
Copy link
Copy Markdown
Contributor

@cyyever cyyever commented Dec 15, 2024

No description provided.

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Dec 15, 2024 •
edited
Loading

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/8814

Note: Links to docs will display an error until the docs builds have been completed.

⏳ No Failures, 4 Pending

As of commit 1ec7215 with merge base 5515dff (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@cyyever cyyever force-pushed the ruff39 branch 2 times, most recently from 036a3da to b505aa5 Compare March 8, 2025 03:43
@AntoineSimoulin
Copy link
Copy Markdown
Member

AntoineSimoulin commented Apr 22, 2025

Hey @cyyever, Thank you for submitting this PR! We're excited about the changes and would like to merge them into our codebase. However, it seems there are a some conflicts that need to be resolved before we can proceed. To streamline the process, I was wondering you could share your workflow and commands used to prepare this PR? This way, we can work together to resolve the conflicts and get the PR ready for merging.
Thanks!

@cyyever
Copy link
Copy Markdown
Contributor Author

cyyever commented Apr 22, 2025

@AntoineSimoulin Fixed

@cyyever cyyever marked this pull request as draft April 23, 2025 00:30
@cyyever cyyever force-pushed the ruff39 branch 2 times, most recently from 8229241 to 215752c Compare April 23, 2025 00:32
@NicolasHug NicolasHug marked this pull request as ready for review April 23, 2025 09:27
NicolasHug
NicolasHug approved these changes Apr 23, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@NicolasHug NicolasHug left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thank you @cyyever !

@NicolasHug NicolasHug merged commit a095de1 into pytorch:main Apr 23, 2025
58 checks passed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 23, 2025

Hey @NicolasHug!

You merged this PR, but no labels were added.
The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

@NicolasHug NicolasHug mentioned this pull request Apr 23, 2025
Merged
@cyyever cyyever deleted the ruff39 branch April 30, 2025 10:08
facebook-github-bot pushed a commit that referenced this pull request Jul 30, 2025
@NicolasHug @facebook-github-bot
[fbsync] Upgrade type hint and others to Python 3.9 (#8814)
0c70082 
Reviewed By: Dan-Flores

Differential Revision: D79175019

fbshipit-source-id: a9b9778af8035fdcb566e02e64730875e4318be9

Co-authored-by: Nicolas Hug <nh.nicolas.hug@gmail.com>
Co-authored-by: Nicolas Hug <contact@nicolas-hug.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NicolasHug NicolasHug NicolasHug approved these changes

Assignees

No one assigned

Labels

cla signed code quality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cyyever @AntoineSimoulin @NicolasHug @facebook-github-bot