-
Notifications
You must be signed in to change notification settings - Fork 24.9k
[CI] update flake8 and mypy lint dependencies #158720
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/158720
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (2 Unrelated Failures)As of commit 98ce675 with merge base aaa384b ( FLAKY - The following job failed but was likely due to flakiness present on trunk:
UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
ghstack-source-id: 00af144 Pull-Request: pytorch#158720
ghstack-source-id: 00af144 Pull-Request: pytorch#158720
ghstack-source-id: 687d6ca Pull-Request: pytorch#158720
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Pull Request resolved: pytorch#158720 Approved by: https://github.com/Skylion007
@pytorchbot revert -m "This broke lint, see https://hud.pytorch.org/hud/pytorch/pytorch/8e57cdb746b4ab28865fdf01532f87b0d21700e9/1?per_page=50&name_filter=lintrunner-noclang" -c nosignal |
@pytorchbot successfully started a revert job. Check the current status here. |
This reverts commit a527e81. Reverted #158720 on behalf of https://github.com/malfet due to This broke lint, see https://hud.pytorch.org/hud/pytorch/pytorch/8e57cdb746b4ab28865fdf01532f87b0d21700e9/1?per_page=50&name_filter=lintrunner-noclang ([comment](#158720 (comment)))
@XuehaiPan your PR has been successfully reverted. |
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
@pytorchbot revert -m "this pr failed internally when build torchgen due to rror: fail: Unknown PyPI project: pyyaml, it seems like this is caused by change PyYAML into pyyaml, please fix it" -c ghfirst |
@pytorchbot successfully started a revert job. Check the current status here. |
Don't want to revert based on edited command |
@pytorchbot revert -m "this pr failed internally when build torchgen due to rror: fail: Unknown PyPI project: pyyaml, it seems like this is caused by change PyYAML into pyyaml, please fix it" -c ghfirst |
@pytorchbot successfully started a revert job. Check the current status here. |
This reverts commit f5130bf. Reverted #158720 on behalf of https://github.com/yangw-dev due to this pr failed internally when build torchgen due to rror: fail: Unknown PyPI project: pyyaml, it seems like this is caused by change PyYAML into pyyaml, please fix it ([comment](#158720 (comment)))
@XuehaiPan your PR has been successfully reverted. |
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: 1 jobs have failed, first few of them are: trunk / linux-jammy-rocm-py3.10 / test (distributed, 1, 1, linux.rocm.gpu.4) Details for Dev Infra teamRaised by workflow job |
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Pull Request resolved: #158720 Approved by: https://github.com/Skylion007
This reverts commit f5130bf. Reverted #158720 on behalf of https://github.com/yangw-dev due to this pr failed internally when build torchgen due to rror: fail: Unknown PyPI project: pyyaml, it seems like this is caused by change PyYAML into pyyaml, please fix it ([comment](#158720 (comment)))
Pull Request resolved: #158720 Approved by: https://github.com/Skylion007
Stack from ghstack (oldest at bottom):