Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CI] Replace flake8 and isort with ruff #48508

Open
Tracked by #47991
MortalHappiness opened this issue Nov 2, 2024 · 2 comments · May be fixed by #49435
Open
Tracked by #47991

[CI] Replace flake8 and isort with ruff #48508

MortalHappiness opened this issue Nov 2, 2024 · 2 comments · May be fixed by #49435
Assignees
@MortalHappiness
Labels
ci-test

Comments

@MortalHappiness
Copy link
Member

MortalHappiness commented Nov 2, 2024 •
edited
Loading

In #47991 (comment)

Since ruff is drop-in replacement for flake8 so upgrading flake8 and switching to ruff should require the similar amount of efforts.
@MortalHappiness MortalHappiness mentioned this issue Nov 2, 2024
Open
25 tasks
@MortalHappiness MortalHappiness self-assigned this Nov 2, 2024
@MortalHappiness MortalHappiness changed the title Replace flake8, isort, black with ruff [CI] Replace flake8, isort, black with ruff Nov 2, 2024
@MortalHappiness MortalHappiness mentioned this issue Nov 12, 2024
Open
@win5923
Copy link
Contributor

win5923 commented Nov 21, 2024 •
edited
Loading

Hi @MortalHappiness, can you provide some guidance on how to replace flake8, isort, and black with ruff? Apart from updating the .pre-commit-config.yaml to use ruff, what else needs to be done?

@win5923 win5923 mentioned this issue Nov 21, 2024
Closed
8 tasks
@MortalHappiness
Copy link
Member Author

Hi @win5923 Actually, I don't know either. You can check the codebase to see what needs to be changed, and if you have any questions, feel free to ping me, and I can ask others for you.

cc @aslonnie

@MortalHappiness MortalHappiness changed the title [CI] Replace flake8, isort, black with ruff [CI] Replace flake8 and isort with ruff Dec 25, 2024
@MortalHappiness MortalHappiness mentioned this issue Dec 25, 2024
Closed
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
[CI][Linter] Migrate to ruff
795f13c 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Fix SyntaxError files
2f6800c 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Fix files autofixable by ruff
2469fd2 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Run ruff pre-commit rule in CI
0eaaadb 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
@MortalHappiness MortalHappiness linked a pull request Dec 25, 2024 that will close this issue
Open
8 tasks
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
[CI][Linter] Migrate to ruff
4f7e608 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Fix SyntaxError files
44a6990 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Fix files autofixable by ruff
c6b6f97 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Run ruff pre-commit rule in CI
18e5b64 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
[CI][Linter] Migrate to ruff
0142c72 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Fix SyntaxError files
1569239 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Run ruff pre-commit rule in CI
3c15dc7 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Update noqa comments to use ruff
0438f43 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Update requirements_compiled.txt
18ead33 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Ignore autofixable rules
3474ec9 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Run ruff pre-commit rule in CI
cb3c3d2 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Update noqa comments to use ruff
9e3c87e 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Update requirements_compiled.txt
0e68ed7 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
Ignore autofixable rules
aab7d03 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 25, 2024
@MortalHappiness
[CI][Linter] Migrate to ruff
78ab98b 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 26, 2024
@MortalHappiness
Update requirements_compiled.txt
bf753df 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 26, 2024
@MortalHappiness
Ignore autofixable rules
19bb4de 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 26, 2024
@MortalHappiness
Pin pre-commit version
44b8174 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
889f5a7 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
4f806c7 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
d66cf7d 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
e685ad8 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
9f19953 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
28a8133 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
15e3b49 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
820708a 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
c193b41 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
df361dc 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
e2ae29c 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
[CI][Linter] Migrate to ruff
938b5e9 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Fix SyntaxError files
c4cff7c 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Run ruff pre-commit rule in CI
de93792 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Update noqa comments to use ruff
0b901f9 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Update requirements_compiled.txt
4538c9e 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Ignore autofixable rules
ebb7d07 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Pin pre-commit version
3d513eb 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 27, 2024
@MortalHappiness
Fix conflicts
7a5cd1a 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
[CI][Linter] Migrate to ruff
7c4c513 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Fix SyntaxError files
ef84628 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Run ruff pre-commit rule in CI
513e7a1 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Update noqa comments to use ruff
2fb2394 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Update requirements_compiled.txt
ed6f223 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Ignore autofixable rules
c319ce0 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Pin pre-commit version
4883242 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness added a commit to MortalHappiness/ray that referenced this issue Dec 28, 2024
@MortalHappiness
Fix conflicts
3e6224f 
Closes: ray-project#48508
Signed-off-by: Chi-Sheng Liu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MortalHappiness MortalHappiness

Labels
ci-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[CI][Linter] Replace flake8 and isort with ruff MortalHappiness/ray
2 participants
@MortalHappiness @win5923