can-gaa-hou
ad32ed83b3
[Fix] Adding missing f prefixes to formatted strings [3/N] ( #164067 )
...
As stated in the title.
* #164068
* __->__ #164067
* #164066
* #164065
Pull Request resolved: https://github.com/pytorch/pytorch/pull/164067
Approved by: https://github.com/Skylion007
2025-09-29 04:35:23 +00:00
Xuehai Pan
178515d0ff
[BE][PYFMT] remove black: finish black -> ruff format migration ( #144557 )
...
Pull Request resolved: https://github.com/pytorch/pytorch/pull/144557
Approved by: https://github.com/ezyang
2025-08-08 07:46:10 +00:00
Xuehai Pan
637e75433c
[BE] always use uv pip if possible in pip_init.py for lintrunner init ( #157199 )
...
Pull Request resolved: https://github.com/pytorch/pytorch/pull/157199
Approved by: https://github.com/ezyang , https://github.com/ZainRizvi
2025-07-21 17:56:05 +00:00
PyTorch MergeBot
cb711c8fa0
Revert "[BE] always use uv pip if possible in pip_init.py for lintrunner init ( #157199 )"
...
This reverts commit 754699610b .
Reverted https://github.com/pytorch/pytorch/pull/157199 on behalf of https://github.com/malfet due to It breaks lintrunner init` for default environments, see https://github.com/pytorch/pytorch/issues/152999 ([comment](https://github.com/pytorch/pytorch/pull/157199#issuecomment-3053279711 ))
2025-07-09 16:26:47 +00:00
Xuehai Pan
754699610b
[BE] always use uv pip if possible in pip_init.py for lintrunner init ( #157199 )
...
Pull Request resolved: https://github.com/pytorch/pytorch/pull/157199
Approved by: https://github.com/ezyang
2025-07-01 06:07:29 +00:00
Nikita Shulga
c10339559d
[BE] Better uv detection in pip init ( #155972 )
...
If one has some UV and non-UV environments locally, one shoudl call `uv
pip install` only on the UV-enabled ones, which could be detected by
checking if `uv/python` path is present in `sys.base_prefix`
Fixes https://github.com/pytorch/pytorch/issues/152999
Pull Request resolved: https://github.com/pytorch/pytorch/pull/155972
Approved by: https://github.com/janeyx99
2025-06-14 01:35:50 +00:00
Nikita Shulga
93efddc67a
Use pip corresponding to python executable ( #141165 )
...
Sometimes `python3` and `pip` are aliased to different runtimes, so it's better to always use `pip3`, but as linter should install packages into the same python environment, it's even better to just call sys.executable with `-mpip install XYZ` arguments
Fixes regression introduced by https://github.com/pytorch/pytorch/pull/124033
Pull Request resolved: https://github.com/pytorch/pytorch/pull/141165
Approved by: https://github.com/Skylion007 , https://github.com/kit1980
2024-11-20 22:58:33 +00:00
Xuehai Pan
8a67daf283
[BE][Easy] enable postponed annotations in tools ( #129375 )
...
Pull Request resolved: https://github.com/pytorch/pytorch/pull/129375
Approved by: https://github.com/malfet
2024-06-29 09:23:35 +00:00
PyTorch MergeBot
a32ce5ce34
Revert "[BE][Easy] enable postponed annotations in tools ( #129375 )"
...
This reverts commit 59eb2897f1 .
Reverted https://github.com/pytorch/pytorch/pull/129375 on behalf of https://github.com/huydhn due to Sorry for reverting your change but I need to revert to cleanly revert https://github.com/pytorch/pytorch/pull/129374 , please do a rebase and reland this ([comment](https://github.com/pytorch/pytorch/pull/129375#issuecomment-2197800541 ))
2024-06-29 00:44:25 +00:00
Xuehai Pan
59eb2897f1
[BE][Easy] enable postponed annotations in tools ( #129375 )
...
Pull Request resolved: https://github.com/pytorch/pytorch/pull/129375
Approved by: https://github.com/malfet
2024-06-28 15:37:54 +00:00
Edward Z. Yang
af9a707233
Use uv in lintrunner init when it is available. ( #124033 )
...
Before, a no-op lintrunner init takes 12s. After, it takes 1s;
a full order of magnitude improvement.
Signed-off-by: Edward Z. Yang <ezyang@meta.com>
Pull Request resolved: https://github.com/pytorch/pytorch/pull/124033
Approved by: https://github.com/cyyever , https://github.com/Skylion007
2024-04-15 14:47:21 +00:00
albanD
b8895df8db
Fix black binary again for debug python ( #86275 )
...
The `--no-binary` flag was not ported when moving from black only to ufmt.
This adds it back.
This is to work around the fact that black binary hard crashes when running with debug python and it needs to be compiled from source.
Pull Request resolved: https://github.com/pytorch/pytorch/pull/86275
Approved by: https://github.com/bdhirsh , https://github.com/malfet
2022-10-05 21:08:40 +00:00
Huy Do
ef035d083e
Add ufmt to unify black and usort ( #81157 )
...
This reworks [80257](https://github.com/pytorch/pytorch/pull/80257 ) a bit to use ufmt:
* ufmt https://ufmt.omnilib.dev/en/stable/ unifies both black and usort to automatically format the file in the "most Pythonic" way
* Also make a demo run for all files in `tools/linter/**/*.py`
Pull Request resolved: https://github.com/pytorch/pytorch/pull/81157
Approved by: https://github.com/suo
2022-07-11 19:20:22 +00:00
albanD
3e0f1a8a32
Add option to skip binaries when doing pip install for lintrunner ( #78668 )
...
This is a workaround for https://github.com/suo/lintrunner/issues/7
Pull Request resolved: https://github.com/pytorch/pytorch/pull/78668
Approved by: https://github.com/suo
2022-06-02 20:50:57 +00:00
Michael Suo
84641d0dba
[lint] fix pip init for lint when user has a global install
...
Closes https://github.com/suo/lintrunner/issues/4 .
Pull Request resolved: https://github.com/pytorch/pytorch/pull/76707
Approved by: https://github.com/janeyx99
2022-05-03 19:36:59 +00:00
Edward Z. Yang
a11c1bbdd0
Run Black on all of tools/
...
Signed-off-by: Edward Z. Yang <ezyangfb.com>
Pull Request resolved: https://github.com/pytorch/pytorch/pull/76089
Approved by: https://github.com/albanD
2022-04-20 17:29:41 +00:00
Michael Suo
1c60b9aaa5
[ci] use lintrunner in CI
...
This changes our lint workflows to use lintrunner for the linters that
are currently supported
+ some random fixes to make things lint clean on master
+ changes to Makefile to use lintrunner
Pull Request resolved: https://github.com/pytorch/pytorch/pull/68460
Approved by: https://github.com/t10-13rocket , https://github.com/seemethere , https://github.com/janeyx99
2022-04-15 00:08:21 +00:00
PyTorch MergeBot
db6165215e
Revert "[ci] use lintrunner in CI"
...
This reverts commit 4c3ee53522 .
Reverted https://github.com/pytorch/pytorch/pull/68460 on behalf of https://github.com/malfet
2022-04-14 23:27:27 +00:00
Michael Suo
4c3ee53522
[ci] use lintrunner in CI
...
This changes our lint workflows to use lintrunner for the linters that
are currently supported
+ some random fixes to make things lint clean on master
+ changes to Makefile to use lintrunner
Pull Request resolved: https://github.com/pytorch/pytorch/pull/68460
Approved by: https://github.com/t10-13rocket , https://github.com/seemethere , https://github.com/janeyx99
2022-04-14 17:43:41 +00:00
Michael Suo
4b021280ad
[lint] add nativefunctions to lintrunner ( #67890 )
...
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67890
Adding another linter. I also added a generic initializer that installs
the right pip packages (you can invoke it by running `lintrunner init`).
Differential Revision:
D32197366
D32197366
Test Plan: Imported from OSS
Reviewed By: driazati
Pulled By: suo
fbshipit-source-id: 82844e78f1ee3047220d8444874eab41d7cc0e9e
2021-11-08 09:44:59 -08:00