c11cdfabd1
ruff is faster and handle everything we had prior. isort configuration done based on the indication from https://github.com/astral-sh/ruff/issues/4670, previousely based on reorder-python-import (#11896) flake8-docstrings was a wrapper around pydocstyle (now archived) that explicitly asks to use ruff in https://github.com/PyCQA/pydocstyle/pull/658. flake8-typing-import is useful mainly for project that support python 3.7 and the one useful check will be implemented in https://github.com/astral-sh/ruff/issues/2302 We need to keep blacken-doc because ruff does not handle detection of python code inside .md and .rst. The direct link to the repo is now used to avoid a redirection. Manual fixes: - Lines that became too long - % formatting that was not done automatically - type: ignore that were moved around - noqa of hard to fix issues (UP031 generally) - fmt: off and fmt: on that is not really identical between black and ruff - autofix re-order in pre-commit from faster to slower Co-authored-by: Ran Benita <ran@unusedvar.com>
85 lines
2.8 KiB
Python
85 lines
2.8 KiB
Python
from __future__ import annotations
|
|
|
|
import functools
|
|
from pathlib import Path
|
|
from typing import Mapping
|
|
import warnings
|
|
|
|
import pluggy
|
|
|
|
from ..compat import LEGACY_PATH
|
|
from ..compat import legacy_path
|
|
from ..deprecated import HOOK_LEGACY_PATH_ARG
|
|
|
|
|
|
# hookname: (Path, LEGACY_PATH)
|
|
imply_paths_hooks: Mapping[str, tuple[str, str]] = {
|
|
"pytest_ignore_collect": ("collection_path", "path"),
|
|
"pytest_collect_file": ("file_path", "path"),
|
|
"pytest_pycollect_makemodule": ("module_path", "path"),
|
|
"pytest_report_header": ("start_path", "startdir"),
|
|
"pytest_report_collectionfinish": ("start_path", "startdir"),
|
|
}
|
|
|
|
|
|
def _check_path(path: Path, fspath: LEGACY_PATH) -> None:
|
|
if Path(fspath) != path:
|
|
raise ValueError(
|
|
f"Path({fspath!r}) != {path!r}\n"
|
|
"if both path and fspath are given they need to be equal"
|
|
)
|
|
|
|
|
|
class PathAwareHookProxy:
|
|
"""
|
|
this helper wraps around hook callers
|
|
until pluggy supports fixingcalls, this one will do
|
|
|
|
it currently doesn't return full hook caller proxies for fixed hooks,
|
|
this may have to be changed later depending on bugs
|
|
"""
|
|
|
|
def __init__(self, hook_relay: pluggy.HookRelay) -> None:
|
|
self._hook_relay = hook_relay
|
|
|
|
def __dir__(self) -> list[str]:
|
|
return dir(self._hook_relay)
|
|
|
|
def __getattr__(self, key: str) -> pluggy.HookCaller:
|
|
hook: pluggy.HookCaller = getattr(self._hook_relay, key)
|
|
if key not in imply_paths_hooks:
|
|
self.__dict__[key] = hook
|
|
return hook
|
|
else:
|
|
path_var, fspath_var = imply_paths_hooks[key]
|
|
|
|
@functools.wraps(hook)
|
|
def fixed_hook(**kw):
|
|
path_value: Path | None = kw.pop(path_var, None)
|
|
fspath_value: LEGACY_PATH | None = kw.pop(fspath_var, None)
|
|
if fspath_value is not None:
|
|
warnings.warn(
|
|
HOOK_LEGACY_PATH_ARG.format(
|
|
pylib_path_arg=fspath_var, pathlib_path_arg=path_var
|
|
),
|
|
stacklevel=2,
|
|
)
|
|
if path_value is not None:
|
|
if fspath_value is not None:
|
|
_check_path(path_value, fspath_value)
|
|
else:
|
|
fspath_value = legacy_path(path_value)
|
|
else:
|
|
assert fspath_value is not None
|
|
path_value = Path(fspath_value)
|
|
|
|
kw[path_var] = path_value
|
|
kw[fspath_var] = fspath_value
|
|
return hook(**kw)
|
|
|
|
fixed_hook.name = hook.name # type: ignore[attr-defined]
|
|
fixed_hook.spec = hook.spec # type: ignore[attr-defined]
|
|
fixed_hook.__name__ = key
|
|
self.__dict__[key] = fixed_hook
|
|
return fixed_hook # type: ignore[return-value]
|