-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Enabled ruff TCH on some files #51827
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
Enabled ruff TCH on some files #51827
Conversation
48fb5e0
to
68635f2
Compare
thanks for your PR - can you run |
…/core/algorithms.py, pandas/core/sorting.py and pandas/core/ops/*. Only for the last two items there were changes needed.
68635f2
to
3e1c1a4
Compare
I ran it and get the following output:
|
thanks @ErdiTk - looks like indeed, it's unmaintained. we should probably just remove it - do you feel like making a separate PR to just remove it? for now, for this PR, you could do
|
No problem, I will create a separate PR to remove the flake8-rst. Indeed seems unmaintained, last update was in 2020. Shall I link it to this PR or are you going to create a new Issue for it? Moreover, the line:
Should be added in the |
linking to this PR seems fine not sure what you mean by your second question, but that doesn't need to go in pyproject.toml |
Sorry I understood now. Will make soon the PR where I remove fake8-rst. |
# Conflicts: # pandas/core/ops/__init__.py # pandas/core/ops/array_ops.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @ErdiTk !
#51740 for the following elements:
No change was required means that the command:
pre-commit run ruff --all-files
had zero errors after removing the relative line from the
pyproject.toml
file without any change (can be tried by doing so in the main branch).