You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would expect D417 to not require *args and **kwargs to be put in the descriptions. Maybe this is not good practice, in which case please let me know.
A minimal code snippet that reproduces the bug.
def f(*args, **kwargs):
"""Description"""
The command you invoked (e.g., ruff /path/to/file.py --fix), ideally including the --isolated flag. pre-commit install with the following config:
Ah yeah, I think this is correct, though it should only be triggering if you include an Args: section in your docstring. If you do document the arguments, I believe it's intended that you also document the *args and **kwargs, and this matches pydocstyle behavior.
If your signature is just*args and **kwargs as above, then it probably makes sense to omit the Args: section altogether -- if I run ruff check --select D417 on the example you provided above, we don't raise an error, which makes sense to me, but are you seeing differently? Here's an example from the playground with that rule selected.
I would expect D417 to not require
*args
and**kwargs
to be put in the descriptions. Maybe this is not good practice, in which case please let me know.ruff /path/to/file.py --fix
), ideally including the--isolated
flag.pre-commit install
with the following config:pyproject.toml
).ruff --version
).The text was updated successfully, but these errors were encountered: