Readd ruff check for scripts, but allow print and missing docstring#316
Conversation
.ruff.toml
Outdated
| "scripts/*" = [ | ||
| "D103", # Don't complain over missing docstrings in scripts | ||
| "T201", # Allow for scripts to write to stdout | ||
| ] |
There was a problem hiding this comment.
I regret placing the [lint.per-file-ignores] above the general [lint] section. Would you be so kind to move this entire section to below ´[lint]` please? I think it's best to have the main rules first in the files.
There was a problem hiding this comment.
I ended up reordering the sections to match the order in https://github.com/home-assistant/core/blob/dev/pyproject.toml.
I also added the pydocstyle section from HA-pyproject, which apparently disables the D401 First line of docstring should be in imperative mood-ruff-error (which I think we have discussed ignoring before at some point)
Also adds the pydocstyle and pytest-style-mark-parentheses = false (both from HA)
sveinse
left a comment
sveinse
left a comment
There was a problem hiding this comment.
I think this looks good. Are we ready to merge?
|
Yep 👍 |
2 participants
No description provided.