mirror of
https://github.com/pre-commit/pre-commit.git
synced 2026-02-17 08:14:42 +04:00
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
This commit is contained in:
parent
9d4ab670d1
commit
917e2102be
24 changed files with 32 additions and 33 deletions
|
|
@ -61,7 +61,7 @@ def filter_by_include_exclude(
|
||||||
names: Iterable[str],
|
names: Iterable[str],
|
||||||
include: str,
|
include: str,
|
||||||
exclude: str,
|
exclude: str,
|
||||||
) -> Generator[str, None, None]:
|
) -> Generator[str]:
|
||||||
include_re, exclude_re = re.compile(include), re.compile(exclude)
|
include_re, exclude_re = re.compile(include), re.compile(exclude)
|
||||||
return (
|
return (
|
||||||
filename for filename in names
|
filename for filename in names
|
||||||
|
|
@ -84,7 +84,7 @@ class Classifier:
|
||||||
types: Iterable[str],
|
types: Iterable[str],
|
||||||
types_or: Iterable[str],
|
types_or: Iterable[str],
|
||||||
exclude_types: Iterable[str],
|
exclude_types: Iterable[str],
|
||||||
) -> Generator[str, None, None]:
|
) -> Generator[str]:
|
||||||
types = frozenset(types)
|
types = frozenset(types)
|
||||||
types_or = frozenset(types_or)
|
types_or = frozenset(types_or)
|
||||||
exclude_types = frozenset(exclude_types)
|
exclude_types = frozenset(exclude_types)
|
||||||
|
|
@ -97,7 +97,7 @@ class Classifier:
|
||||||
):
|
):
|
||||||
yield filename
|
yield filename
|
||||||
|
|
||||||
def filenames_for_hook(self, hook: Hook) -> Generator[str, None, None]:
|
def filenames_for_hook(self, hook: Hook) -> Generator[str]:
|
||||||
return self.by_types(
|
return self.by_types(
|
||||||
filter_by_include_exclude(
|
filter_by_include_exclude(
|
||||||
self.filenames,
|
self.filenames,
|
||||||
|
|
|
||||||
|
|
@ -33,7 +33,7 @@ def format_env(parts: SubstitutionT, env: MutableMapping[str, str]) -> str:
|
||||||
def envcontext(
|
def envcontext(
|
||||||
patch: PatchesT,
|
patch: PatchesT,
|
||||||
_env: MutableMapping[str, str] | None = None,
|
_env: MutableMapping[str, str] | None = None,
|
||||||
) -> Generator[None, None, None]:
|
) -> Generator[None]:
|
||||||
"""In this context, `os.environ` is modified according to `patch`.
|
"""In this context, `os.environ` is modified according to `patch`.
|
||||||
|
|
||||||
`patch` is an iterable of 2-tuples (key, value):
|
`patch` is an iterable of 2-tuples (key, value):
|
||||||
|
|
|
||||||
|
|
@ -68,7 +68,7 @@ def _log_and_exit(
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def error_handler() -> Generator[None, None, None]:
|
def error_handler() -> Generator[None]:
|
||||||
try:
|
try:
|
||||||
yield
|
yield
|
||||||
except (Exception, KeyboardInterrupt) as e:
|
except (Exception, KeyboardInterrupt) as e:
|
||||||
|
|
|
||||||
|
|
@ -20,7 +20,7 @@ if sys.platform == 'win32': # pragma: no cover (windows)
|
||||||
def _locked(
|
def _locked(
|
||||||
fileno: int,
|
fileno: int,
|
||||||
blocked_cb: Callable[[], None],
|
blocked_cb: Callable[[], None],
|
||||||
) -> Generator[None, None, None]:
|
) -> Generator[None]:
|
||||||
try:
|
try:
|
||||||
msvcrt.locking(fileno, msvcrt.LK_NBLCK, _region)
|
msvcrt.locking(fileno, msvcrt.LK_NBLCK, _region)
|
||||||
except OSError:
|
except OSError:
|
||||||
|
|
@ -53,7 +53,7 @@ else: # pragma: win32 no cover
|
||||||
def _locked(
|
def _locked(
|
||||||
fileno: int,
|
fileno: int,
|
||||||
blocked_cb: Callable[[], None],
|
blocked_cb: Callable[[], None],
|
||||||
) -> Generator[None, None, None]:
|
) -> Generator[None]:
|
||||||
try:
|
try:
|
||||||
fcntl.flock(fileno, fcntl.LOCK_EX | fcntl.LOCK_NB)
|
fcntl.flock(fileno, fcntl.LOCK_EX | fcntl.LOCK_NB)
|
||||||
except OSError: # pragma: no cover (tests are single-threaded)
|
except OSError: # pragma: no cover (tests are single-threaded)
|
||||||
|
|
@ -69,7 +69,7 @@ else: # pragma: win32 no cover
|
||||||
def lock(
|
def lock(
|
||||||
path: str,
|
path: str,
|
||||||
blocked_cb: Callable[[], None],
|
blocked_cb: Callable[[], None],
|
||||||
) -> Generator[None, None, None]:
|
) -> Generator[None]:
|
||||||
with open(path, 'a+') as f:
|
with open(path, 'a+') as f:
|
||||||
with _locked(f.fileno(), blocked_cb):
|
with _locked(f.fileno(), blocked_cb):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -127,7 +127,7 @@ def no_install(
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def no_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def no_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
yield
|
yield
|
||||||
|
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -41,7 +41,7 @@ def get_env_patch(env: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -70,7 +70,7 @@ def get_env_patch(target_dir: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -29,7 +29,7 @@ def get_env_patch(venv: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -30,14 +30,14 @@ def get_env_patch(venv: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def _nuget_config_no_sources() -> Generator[str, None, None]:
|
def _nuget_config_no_sources() -> Generator[str]:
|
||||||
with tempfile.TemporaryDirectory() as tmpdir:
|
with tempfile.TemporaryDirectory() as tmpdir:
|
||||||
nuget_config = os.path.join(tmpdir, 'nuget.config')
|
nuget_config = os.path.join(tmpdir, 'nuget.config')
|
||||||
with open(nuget_config, 'w') as f:
|
with open(nuget_config, 'w') as f:
|
||||||
|
|
|
||||||
|
|
@ -121,7 +121,7 @@ def _install_go(version: str, dest: str) -> None:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir, version)):
|
with envcontext(get_env_patch(envdir, version)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -24,7 +24,7 @@ def get_env_patch(target_dir: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -44,7 +44,7 @@ def get_env_patch(d: str) -> PatchesT: # pragma: win32 no cover
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager # pragma: win32 no cover
|
@contextlib.contextmanager # pragma: win32 no cover
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -59,7 +59,7 @@ def get_env_patch(venv: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -33,7 +33,7 @@ def get_env_patch(venv: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -152,7 +152,7 @@ def norm_version(version: str) -> str | None:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -85,7 +85,7 @@ def health_check(prefix: Prefix, version: str) -> str | None:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def _r_code_in_tempfile(code: str) -> Generator[str, None, None]:
|
def _r_code_in_tempfile(code: str) -> Generator[str]:
|
||||||
"""
|
"""
|
||||||
To avoid quoting and escaping issues, avoid `Rscript [options] -e {expr}`
|
To avoid quoting and escaping issues, avoid `Rscript [options] -e {expr}`
|
||||||
but use `Rscript [options] path/to/file_with_expr.R`
|
but use `Rscript [options] path/to/file_with_expr.R`
|
||||||
|
|
@ -105,7 +105,7 @@ def get_env_patch(venv: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -73,7 +73,7 @@ def get_env_patch(
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir, version)):
|
with envcontext(get_env_patch(envdir, version)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -61,7 +61,7 @@ def get_env_patch(target_dir: str, version: str) -> PatchesT:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir, version)):
|
with envcontext(get_env_patch(envdir, version)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -27,7 +27,7 @@ def get_env_patch(venv: str) -> PatchesT: # pragma: win32 no cover
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager # pragma: win32 no cover
|
@contextlib.contextmanager # pragma: win32 no cover
|
||||||
def in_env(prefix: Prefix, version: str) -> Generator[None, None, None]:
|
def in_env(prefix: Prefix, version: str) -> Generator[None]:
|
||||||
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
envdir = lang_base.environment_dir(prefix, ENVIRONMENT_DIR, version)
|
||||||
with envcontext(get_env_patch(envdir)):
|
with envcontext(get_env_patch(envdir)):
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -32,7 +32,7 @@ class LoggingHandler(logging.Handler):
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def logging_handler(use_color: bool) -> Generator[None, None, None]:
|
def logging_handler(use_color: bool) -> Generator[None]:
|
||||||
handler = LoggingHandler(use_color)
|
handler = LoggingHandler(use_color)
|
||||||
logger.addHandler(handler)
|
logger.addHandler(handler)
|
||||||
logger.setLevel(logging.INFO)
|
logger.setLevel(logging.INFO)
|
||||||
|
|
|
||||||
|
|
@ -33,7 +33,7 @@ def _git_apply(patch: str) -> None:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def _intent_to_add_cleared() -> Generator[None, None, None]:
|
def _intent_to_add_cleared() -> Generator[None]:
|
||||||
intent_to_add = git.intent_to_add_files()
|
intent_to_add = git.intent_to_add_files()
|
||||||
if intent_to_add:
|
if intent_to_add:
|
||||||
logger.warning('Unstaged intent-to-add files detected.')
|
logger.warning('Unstaged intent-to-add files detected.')
|
||||||
|
|
@ -48,7 +48,7 @@ def _intent_to_add_cleared() -> Generator[None, None, None]:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def _unstaged_changes_cleared(patch_dir: str) -> Generator[None, None, None]:
|
def _unstaged_changes_cleared(patch_dir: str) -> Generator[None]:
|
||||||
tree = cmd_output('git', 'write-tree')[1].strip()
|
tree = cmd_output('git', 'write-tree')[1].strip()
|
||||||
diff_cmd = (
|
diff_cmd = (
|
||||||
'git', 'diff-index', '--ignore-submodules', '--binary',
|
'git', 'diff-index', '--ignore-submodules', '--binary',
|
||||||
|
|
@ -105,7 +105,7 @@ def _unstaged_changes_cleared(patch_dir: str) -> Generator[None, None, None]:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def staged_files_only(patch_dir: str) -> Generator[None, None, None]:
|
def staged_files_only(patch_dir: str) -> Generator[None]:
|
||||||
"""Clear any unstaged changes from the git working directory inside this
|
"""Clear any unstaged changes from the git working directory inside this
|
||||||
context.
|
context.
|
||||||
"""
|
"""
|
||||||
|
|
|
||||||
|
|
@ -101,7 +101,7 @@ class Store:
|
||||||
os.replace(tmpfile, self.db_path)
|
os.replace(tmpfile, self.db_path)
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def exclusive_lock(self) -> Generator[None, None, None]:
|
def exclusive_lock(self) -> Generator[None]:
|
||||||
def blocked_cb() -> None: # pragma: no cover (tests are in-process)
|
def blocked_cb() -> None: # pragma: no cover (tests are in-process)
|
||||||
logger.info('Locking pre-commit directory')
|
logger.info('Locking pre-commit directory')
|
||||||
|
|
||||||
|
|
@ -112,7 +112,7 @@ class Store:
|
||||||
def connect(
|
def connect(
|
||||||
self,
|
self,
|
||||||
db_path: str | None = None,
|
db_path: str | None = None,
|
||||||
) -> Generator[sqlite3.Connection, None, None]:
|
) -> Generator[sqlite3.Connection]:
|
||||||
db_path = db_path or self.db_path
|
db_path = db_path or self.db_path
|
||||||
# sqlite doesn't close its fd with its contextmanager >.<
|
# sqlite doesn't close its fd with its contextmanager >.<
|
||||||
# contextlib.closing fixes this.
|
# contextlib.closing fixes this.
|
||||||
|
|
|
||||||
|
|
@ -25,7 +25,7 @@ def force_bytes(exc: Any) -> bytes:
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def clean_path_on_failure(path: str) -> Generator[None, None, None]:
|
def clean_path_on_failure(path: str) -> Generator[None]:
|
||||||
"""Cleans up the directory on an exceptional failure."""
|
"""Cleans up the directory on an exceptional failure."""
|
||||||
try:
|
try:
|
||||||
yield
|
yield
|
||||||
|
|
|
||||||
|
|
@ -120,7 +120,6 @@ def partition(
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
def _thread_mapper(maxsize: int) -> Generator[
|
def _thread_mapper(maxsize: int) -> Generator[
|
||||||
Callable[[Callable[[TArg], TRet], Iterable[TArg]], Iterable[TRet]],
|
Callable[[Callable[[TArg], TRet], Iterable[TArg]], Iterable[TRet]],
|
||||||
None, None,
|
|
||||||
]:
|
]:
|
||||||
if maxsize == 1:
|
if maxsize == 1:
|
||||||
yield map
|
yield map
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue