-
Notifications
You must be signed in to change notification settings - Fork 685
Mass migrate to runtime.python_library #14153
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/14153
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New Failure, 5 Unrelated FailuresAs of commit ef79a05 with merge base 53b7ec5 ( NEW FAILURE - The following job has failed:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
BROKEN TRUNK - The following jobs failed but was 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. |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
we'll want to land this internally first so it goes through checks; please don't merge it here |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
2 similar comments
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
1 similar comment
@swolchok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Differential Revision: D82127662 Pull Request resolved: pytorch#14153
We are supposed to be using this everywhere IIUC, but there were a bunch of files not doing it. Mass find/replace.
Differential Revision: D82127662