[pre-commit.ci] pre-commit autoupdate (#30)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Tobias Raabe <[email protected]>

Author: pre-commit-ci[bot]

.github/workflows/main.yml

@@ -27,7 +27,7 @@ jobs:
       fail-fast: false
       matrix:
         os: ['ubuntu-latest', 'macos-latest', 'windows-latest']
-        python-version: ['3.7', '3.8', '3.9', '3.10', '3.11']
+        python-version: ['3.8', '3.9', '3.10', '3.11']
 
     steps:
       - uses: actions/checkout@v4

.pre-commit-config.yaml

@@ -27,20 +27,20 @@ repos:
     -   id: python-no-log-warn
     -   id: python-use-type-annotations
     -   id: text-unicode-replacement-char
--   repo: https://github.com/asottile/reorder_python_imports
-    rev: v3.9.0
+-   repo: https://github.com/asottile/reorder-python-imports
+    rev: v3.12.0
     hooks:
     -   id: reorder-python-imports
         args:
-        - --py37-plus
+        - --py38-plus
         - --add-import
         - from __future__ import annotations
 -   repo: https://github.com/asottile/setup-cfg-fmt
-    rev: v2.2.0
+    rev: v2.5.0
     hooks:
     -   id: setup-cfg-fmt
 -   repo: https://github.com/PyCQA/docformatter
-    rev: v1.6.0.rc1
+    rev: v1.7.5
     hooks:
     -   id: docformatter
         args:
@@ -51,15 +51,15 @@ repos:
         - '88'
         - --blank
 -   repo: https://github.com/psf/black
-    rev: 22.12.0
+    rev: 23.9.1
     hooks:
     -   id: black
--   repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.228
+-   repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.0.292
     hooks:
     -   id: ruff
 -   repo: https://github.com/dosisod/refurb
-    rev: v1.10.0
+    rev: v1.21.0
     hooks:
     -   id: refurb
         args:
@@ -75,7 +75,7 @@ repos:
         - src
         - tests
 -   repo: https://github.com/executablebooks/mdformat
-    rev: 0.7.16
+    rev: 0.7.17
     hooks:
     -   id: mdformat
         additional_dependencies:
@@ -85,11 +85,11 @@ repos:
         - --wrap
         - '88'
 -   repo: https://github.com/codespell-project/codespell
-    rev: v2.2.2
+    rev: v2.2.6
     hooks:
     -   id: codespell
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.991
+    rev: v1.5.1
     hooks:
     -   id: mypy
         args:

environment.yml

@@ -5,14 +5,14 @@ channels:
   - nodefaults
 
 dependencies:
-  - python >=3.7
+  - python >=3.8
   - pip
   - setuptools_scm
   - toml
 
   # Package dependencies
-  - pytask >=0.3
-  - pytask-parallel >=0.3
+  - pytask <0.4
+  - pytask-parallel <0.4
 
   # Misc
   - black

pyproject.toml

@@ -25,10 +25,12 @@ ignore_errors = true
 
 
 [tool.ruff]
-target-version = "py37"
+target-version = "py38"
 select = ["ALL"]
 fix = true
 extend-ignore = [
+    "I",  # ignore isort
+    "TRY",
     # Numpy docstyle
     "D107",
     "D203",
@@ -44,7 +46,6 @@ extend-ignore = [
     "RET504",  # unnecessary variable assignment before return.
     "S101",  # raise errors for asserts.
     "B905",  # strict parameter for zip that was implemented in py310.
-    "I",  # ignore isort
     "ANN101",  # type annotating self
     "ANN102",  # type annotating cls
     "FBT",  # flake8-boolean-trap
@@ -62,3 +63,15 @@ extend-ignore = [
 
 [tool.ruff.pydocstyle]
 convention = "numpy"
+
+
+[tool.pytest.ini_options]
+# Do not add src since it messes with the loading of pytask-parallel as a plugin.
+testpaths = ["tests"]
+markers = [
+    "wip: Tests that are work-in-progress.",
+    "unit: Flag for unit tests which target mainly a single function.",
+    "integration: Flag for integration tests which may comprise of multiple unit tests.",
+    "end_to_end: Flag for tests that cover the whole program.",
+]
+norecursedirs = [".idea", ".tox"]

setup.cfg

@@ -7,7 +7,7 @@ url = https://github.com/pytask-dev/pytask-stata
 author = Tobias Raabe
 author_email = [email protected]
 license = MIT
-license_file = LICENSE
+license_files = LICENSE
 platforms = any
 classifiers =
     Development Status :: 4 - Beta
@@ -26,7 +26,7 @@ packages = find:
 install_requires =
     click
     pytask>=0.3
-python_requires = >=3.7
+python_requires = >=3.8
 include_package_data = True
 package_dir = =src
 zip_safe = False

src/pytask_stata/collect.py

@@ -3,9 +3,9 @@
 
 import functools
 import subprocess
-from pathlib import Path
 from types import FunctionType
 from typing import Any
+from typing import TYPE_CHECKING
 
 from pytask import depends_on
 from pytask import has_mark
@@ -19,14 +19,17 @@
 from pytask_stata.shared import convert_task_id_to_name_of_log_file
 from pytask_stata.shared import stata
 
+if TYPE_CHECKING:
+    from pathlib import Path
+
 
 def run_stata_script(
     executable: str, script: Path, options: list[str], log_name: list[str], cwd: Path
 ) -> None:
     """Run an R script."""
     cmd = [executable, "-e", "do", script.as_posix(), *options, *log_name]
     print("Executing " + " ".join(cmd) + ".")  # noqa: T201
-    subprocess.run(cmd, cwd=cwd, check=True)
+    subprocess.run(cmd, cwd=cwd, check=True)  # noqa: S603
 
 
 @hookimpl

src/pytask_stata/plugin.py

@@ -1,14 +1,18 @@
 """Register hook specifications and implementations."""
 from __future__ import annotations
 
+from typing import TYPE_CHECKING
+
 from _pytask.config import hookimpl
-from pluggy import PluginManager
 from pytask_stata import cli
 from pytask_stata import collect
 from pytask_stata import config
 from pytask_stata import execute
 from pytask_stata import parametrize
 
+if TYPE_CHECKING:
+    from pluggy import PluginManager
+
 
 @hookimpl
 def pytask_add_hooks(pm: PluginManager) -> None:

src/pytask_stata/shared.py

@@ -2,10 +2,13 @@
 from __future__ import annotations
 
 import sys
-from pathlib import Path
 from typing import Any
 from typing import Iterable
 from typing import Sequence
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    from pathlib import Path
 
 
 if sys.platform == "darwin":

tox.ini

@@ -1,51 +1,17 @@
 [tox]
 envlist = pytest
-skipsdist = True
-skip_missing_interpreters = True
-passenv = PY_IGNORE_IMPORTMISMATCH
-whitelist_externals = python
 
 [testenv]
-basepython = python
+usedevelop = true
 
 [testenv:pytest]
 conda_channels =
     conda-forge
     nodefaults
 conda_deps =
-    pytask >=0.3
+    pytask =0.3
     pytest
     pytest-cov
     pytest-xdist
 commands =
-    pip install --no-deps -e .
     pytest {posargs}
-
-[flake8]
-docstring-convention = numpy
-ignore =
-    D
-    E203  ; ignores whitespace around : which is enforced by Black.
-    W503  ; ignores linebreak before binary operator which is enforced by Black.
-    PT006 ; ignores that parametrizing tests with tuple argument names is preferred.
-    PT023 ; ignores parentheses at the end of markers.
-max-line-length = 88
-warn-symbols =
-    pytest.mark.wip = Remove 'wip' flag for tests.
-    pytest.mark.skip = Remove 'skip' flag for tests.
-
-[pytest]
-addopts = --doctest-modules
-filterwarnings =
-    ignore: the imp module is deprecated in favour of importlib
-    ignore: Using or importing the ABCs from 'collections' instead of
-    ignore: The (parser|symbol) module is deprecated and will be removed in future
-junit_family = xunit2
-markers =
-    wip: Tests that are work-in-progress.
-    unit: Flag for unit tests which target mainly a single function.
-    integration: Flag for integration tests which may comprise of multiple unit tests.
-    end_to_end: Flag for tests that cover the whole program.
-norecursedirs =
-    .idea
-    .tox