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.

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

Fix auto-fixable pydocstyle errors through ruff #2850

Merged
merged 6 commits into from
Feb 19, 2023
Merged

Conversation

janosh
Copy link
Member

@janosh janosh commented Feb 19, 2023

Would be good in the long run to enable pydocstyle (rule class "D" in ruff) across the whole code base but at the moment, there are too many existing errors to fix them all. This is just a swing at fixing all the auto-fixable ones with

ruff check . --fix-only --select D

ruff 0.0.247
docs_rst/conf-docset.py:1:1: I002 [*] Missing required import: `from __future__ import annotations`
docs_rst/conf-normal.py:1:1: I002 [*] Missing required import: `from __future__ import annotations`
docs_rst/conf.py:1:1: I002 [*] Missing required import: `from __future__ import annotations`
test_files/Zr_Vasprun/test_vasprun.py:1:1: I002 [*] Missing required import: `from __future__ import annotations`
Found 4 errors.
[*] 4 potentially fixable with the --fix option.
@janosh janosh enabled auto-merge (squash) February 19, 2023 16:30
@janosh janosh merged commit ebec849 into master Feb 19, 2023
@janosh janosh deleted the ruff-pydocstyle branch February 19, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant