Narrow PERF ruleset

khanlab · Dec 13, 2023 · dfe4334 · dfe4334
1 parent db0125a
commit dfe4334
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -166,12 +166,13 @@ select = [
     # "PTH",  # flake8-use-pathlib
     "TRY",  # tryceratops
     "FLY",  # flynt
-    "PERF", # Perflint
     "D",    # pydocstyle
     "PL",   # pylint
     "RUF",  # ruff
     "T100", # debugger
     "I",    # isort
+    # Other perf rules give more false positives than benefits
+    "PERF101", # do not cast iterable to list
 ]
 ignore = [
     "PLR0913",

diff --git a/snakebids/core/input_generation.py b/snakebids/core/input_generation.py
@@ -451,8 +451,7 @@ def write_derivative_json(snakemake: Snakemake, **kwargs: dict[str, Any]) -> Non
         This function requires input.json and output.json to be defined, as
         it will read and write json files
     """
-    with open(snakemake.input.json, encoding="utf-8") as input_json:
-        sidecar = json.load(input_json)
+    sidecar = json.loads(Path(snakemake.input.json).read_text())
 
     sidecar.update(
         {