ruff: enable SIM115: Use context handler for opening files

.github/workflows/issues.py

@@ -159,9 +159,10 @@ def get_repo():
 
 @functools.lru_cache()
 def _get_event_data():
-    ret = json.load(open(os.environ["GITHUB_EVENT_PATH"]))
-    pp(ret)
-    return ret
+    with open(open(os.environ["GITHUB_EVENT_PATH"])) as f:
+        ret = json.load(f)
+        pp(ret)
+        return ret
 
 
 def is_event_new_issue():

psutil/_common.py

@@ -753,7 +753,7 @@ def open_text(fname):
     # See:
     # https://github.com/giampaolo/psutil/issues/675
     # https://github.com/giampaolo/psutil/pull/733
-    fobj = open(
+    fobj = open(  # noqa: SIM115
         fname,
         buffering=FILE_READ_BUFFER_SIZE,
         encoding=ENCODING,

psutil/tests/__init__.py

@@ -263,7 +263,7 @@ def attempt(exe):
 
 
 PYTHON_EXE, PYTHON_EXE_ENV = _get_py_exe()
-DEVNULL = open(os.devnull, 'r+')
+DEVNULL = open(os.devnull, 'r+')  # noqa: SIM115
 atexit.register(DEVNULL.close)
 
 VALID_PROC_STATUSES = [

psutil/tests/test_process.py

@@ -1119,7 +1119,7 @@ def test_num_fds(self):
         p = psutil.Process()
         testfn = self.get_testfn()
         start = p.num_fds()
-        file = open(testfn, 'w')
+        file = open(testfn, 'w')  # noqa: SIM115
         self.addCleanup(file.close)
         assert p.num_fds() == start + 1
         sock = socket.socket()

pyproject.toml

@@ -84,7 +84,6 @@ ignore = [
     "S",  # flake8-bandit
     "SIM102",  # Use a single `if` statement instead of nested `if` statements
     "SIM105",  # Use `contextlib.suppress(OSError)` instead of `try`-`except`-`pass`
-    "SIM115",  # Use context handler for opening files
     "SIM117",  # Use a single `with` statement with multiple contexts instead of nested `with` statements
     "SLF",  # flake8-self
     "TD",  # all TODOs, XXXs, etc.