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

remove redundant typing casts #8140

Merged
merged 2 commits into from
Dec 4, 2023
Merged

remove redundant typing casts #8140

merged 2 commits into from
Dec 4, 2023

Conversation

pmeier
Copy link
Collaborator

@pmeier pmeier commented Dec 4, 2023

They are no longer needed as of mypy==1.7.1. First failure on main: https://github.com/pytorch/vision/actions/runs/6973442404/job/18977451632. On the PR (#8134) we still installed mypy==1.7.0 and thus this was not visible: https://github.com/pytorch/vision/actions/runs/6972043770/job/18973409450?pr=8134.

@pmeier pmeier added the other if you have no clue or if you will manually handle the PR in the release notes label Dec 4, 2023
@pmeier pmeier requested a review from NicolasHug December 4, 2023 10:14
Copy link

pytorch-bot bot commented Dec 4, 2023

🔗 Helpful Links

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

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

✅ You can merge normally! (8 Unrelated Failures)

As of commit 9dfa237 with merge base c1e2095 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

@pmeier
Copy link
Collaborator Author

pmeier commented Dec 4, 2023

@pmeier pmeier merged commit 3feb502 into pytorch:main Dec 4, 2023
54 of 62 checks passed
@pmeier pmeier deleted the lint branch December 4, 2023 10:45
Copy link

github-actions bot commented Dec 4, 2023

Hey @pmeier!

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

facebook-github-bot pushed a commit that referenced this pull request Jan 16, 2024
Reviewed By: vmoens

Differential Revision: D52539013

fbshipit-source-id: 719af2000a4d4c66118a69893a005c037beb5177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed other if you have no clue or if you will manually handle the PR in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants