qa: Use context managers where it makes sense

Spotted by the latest version of pylint.

src/cogite/checks/pre_merge.py

@@ -16,9 +16,9 @@
 
 
 def run_command(command):
-    process = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE)
-    process.wait()
-    return b"".join(process.stdout.readlines()).decode(sys.stdout.encoding)
+    with subprocess.Popen(command, shell=True, stdout=subprocess.PIPE) as process:
+        process.wait()
+        return b"".join(process.stdout.readlines()).decode(sys.stdout.encoding)
 
 
 def check_commits(

src/cogite/interaction.py

@@ -22,11 +22,11 @@ def input_from_file(starting_text=''):
         path = pathlib.Path(tmp.name)
         path.write_text(starting_text)
 
-        process = subprocess.Popen(f"$EDITOR {path}", shell=True)
-        process.wait()
+        with subprocess.Popen(f"$EDITOR {path}", shell=True) as process:
+            process.wait()
 
-        if process.returncode != os.EX_OK:
-            return None
+            if process.returncode != os.EX_OK:
+                return None
 
         return path.read_text()
 

src/cogite/requests.py

@@ -43,6 +43,7 @@ def send(method, url, query=None, data=None, json=None, headers=None):
         method=method,
     )
     try:
+        # pylint: disable=consider-using-with
         return urllib.request.urlopen(request, timeout=TIMEOUT)
     except urllib.error.HTTPError as exc:
         content = exc.file.read().decode('utf-8')