Restore the workflow to update vision repo viable/strict #9274
Conversation
Signed-off-by: Huy Do <huydhn@gmail.com>
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/9274
Note: Links to docs will display an error until the docs builds have been completed. This comment was automatically generated by Dr. CI and updates every 15 minutes. |
| with: | ||
| repository: pytorch/vision | ||
| stable-branch: viable/strict | ||
| requires: '[\"Build Linux\", \"Build M1\", \"Build Macos\", \"Build Windows\", \"Tests\", \"CMake\", \"Lint\", \"Docs\"]' |
There was a problem hiding this comment.
Can we just remove the macOS + Windows stuff here?
Is it really necessary?
There was a problem hiding this comment.
Yeah, let me tentatively make that change unless @NicolasHug wants to keep them. MacOS (x86) is dead and should be removed anyway, but Windows is still a thing from what I see in https://github.com/pytorch/vision/actions
Signed-off-by: Huy Do <huydhn@gmail.com>
NicolasHug
left a comment
NicolasHug
left a comment
There was a problem hiding this comment.
I slightly updated the requires list - thank you so much for re-enabling this @huydhn !
|
Hey @NicolasHug! You merged this PR, but no labels were added. |
Copy from https://github.com/pytorch/executorch/blob/main/.github/workflows/update-viablestrict.yml as used by ExecuTorch:
requireschecks from Remove viable strict workflow #8676. @NicolasHug Please double check that they are what vision needs to promoteviable/strict. All the signals there need to be green for the promotion to happen.update-viable-strictenv and populate with secrets for PyTorch update bot and ClickHouse.