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

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0)
- [github.com/codespell-project/codespell: v2.2.6 → v2.3.0](codespell-project/codespell@v2.2.6...v2.3.0)
- [github.com/asottile/pyupgrade: v3.15.1 → v3.15.2](asottile/pyupgrade@v3.15.1...v3.15.2)
- [github.com/psf/black: 24.3.0 → 24.4.2](psf/black@24.3.0...24.4.2)
- [github.com/pre-commit/mirrors-mypy: v1.9.0 → v1.10.0](pre-commit/mirrors-mypy@v1.9.0...v1.10.0)

* Update test_initial.py - linting line 878

Signed-off-by: Jim-smith <jim-smith@users.noreply.github.com>

* Update acro_stata_parser.py linting for line 267

Signed-off-by: Jim-smith <jim-smith@users.noreply.github.com>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Signed-off-by: Jim-smith <jim-smith@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Jim-smith <jim-smith@users.noreply.github.com>

.pre-commit-config.yaml

@@ -8,7 +8,7 @@ repos:
 
   # Standard hooks
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v4.6.0
     hooks:
       - id: check-merge-conflict
       - id: end-of-file-fixer
@@ -26,7 +26,7 @@ repos:
 
   # Check for spelling
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.2.6
+    rev: v2.3.0
     hooks:
       - id: codespell
         args: ["-L", "tre"]
@@ -39,7 +39,7 @@ repos:
 
   # Upgrade old Python syntax
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.15.1
+    rev: v3.15.2
     hooks:
       - id: pyupgrade
         args: [--py310-plus]
@@ -66,7 +66,7 @@ repos:
 
   # Black format Python and notebooks
   - repo: https://github.com/psf/black
-    rev: 24.3.0
+    rev: 24.4.2
     hooks:
       - id: black-jupyter
 
@@ -84,7 +84,7 @@ repos:
 
   # Check types with mypy
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.9.0
+    rev: v1.10.0
     hooks:
       - id: mypy
         args: ["--ignore-missing-imports", "--follow-imports", "skip", "--check-untyped-defs"]

acro/acro_stata_parser.py

@@ -288,6 +288,7 @@ def run_output_command(command: str, varlist: list) -> str:
     first element of varlist is output affected
     rest (if relevant) is string passed to command.
     """
+    outcome = ""
     if len(varlist) == 0:
         return "syntax error: please pass the name of the output to be changed"
 

test/test_initial.py

@@ -875,7 +875,7 @@ def test_crosstab_multiple_aggregate_function(data, acro):
     assert (
         output.summary == correct_summary
     ), f"\n{output.summary}\n should be \n{correct_summary}\n"
-    print(f"{output.output[0]['mean'][ 'R/G'].sum()}")
+    print(f"{output.output[0]['mean']['R/G'].sum()}")
     correctval = 97383496.0
     errmsg = f"{output.output[0]['mean']['R/G'].sum()} should be {correctval}"
     assert correctval == output.output[0]["mean"]["R/G"].sum(), errmsg