chore: Avoid unnecessary assignment before for-loop

open-contracting · Sep 28, 2024 · a77fac5 · a77fac5
1 parent 875b39d
commit a77fac5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/ocdsindex/__main__.py b/ocdsindex/__main__.py
@@ -123,8 +123,7 @@ def copy(host, source, destination):
 
         for index_result in es.cat.indices(format="json"):
             index = index_result["index"]
-            result = es.search(index=index, size=10000, query={"term": {"base_url": source}})
-            for hit in result["hits"]["hits"]:
+            for hit in es.search(index=index, size=10000, query={"term": {"base_url": source}})["hits"]["hits"]:
                 document = hit["_source"]
                 for field in ("url", "base_url"):
                     document[field] = document[field].replace(source, destination)

diff --git a/pyproject.toml b/pyproject.toml
@@ -48,7 +48,7 @@ target-version = "py39"
 select = ["ALL"]
 ignore = [
     "ANN", "C901", "COM812", "D203", "D212", "D415", "EM", "ISC001", "PERF203", "PLR091", "Q000",
-    "D1",
+    "D1", "D205",
     "PLR2004",  # magic
     "PTH",
 ]