-
Notifications
You must be signed in to change notification settings - Fork 24.2k
[CI] Run lintrunner
on generated .pyi
stub files in CI
#129887
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/129887
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 New FailuresAs of commit 818450e with merge base 09c4da9 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@ezyang How about raising warnings rather than errors for these files? See my comment at #129420 (comment). |
lintrunner
on generated .pyi
stub fileslintrunner
on generated .pyi
stub files
ghstack-source-id: d78a77f Pull Request resolved: pytorch#129887
ghstack-source-id: 934e29f Pull Request resolved: pytorch#129887
ghstack-source-id: 767f5b1 Pull Request resolved: pytorch#129887
See #129420 (comment), this PR is aimed at preventing potential typing errors in the future.
I have fixed all lint errors on the current generated
.pyi
stub files.Stack from ghstack (oldest at bottom):
lintrunner
on generated.pyi
stub files in CI #129887torch/utils/data/datapipes/gen_pyi.py
withtorchgen
#150626__all__
totorch/nn/functional.pyi
andtorch/return_types.pyi
#129872.pyi
stub files #129420torchgen.utils.FileManager
to acceptpathlib.Path
#129871cc @ezyang @malfet @xuzhao9 @gramster @seemethere @pytorch/pytorch-dev-infra