Some ruff fixes

Author: ianthomas23

docs/conf.py

@@ -76,7 +76,7 @@
 here = Path(__file__).parent.resolve()
 version_py = Path(here) / os.pardir / "ipykernel" / "_version.py"
 with open(version_py) as f:
-    exec(compile(f.read(), version_py, "exec"), version_ns)
+    exec(compile(f.read(), version_py, "exec"), version_ns)  # noqa: S102
 
 # The short X.Y version.
 version = "%i.%i" % version_ns["version_info"][:2]

ipykernel/connect.py

@@ -116,7 +116,7 @@ def connect_qtconsole(
 
     cf = _find_connection_file(connection_file)
 
-    cmd = ";".join(["from qtconsole import qtconsoleapp", "qtconsoleapp.main()"])
+    cmd = "from qtconsole import qtconsoleapp; qtconsoleapp.main()"
 
     kwargs: dict[str, Any] = {}
     # Launch the Qt console in a separate session & process group, so

ipykernel/debugger.py

@@ -39,7 +39,7 @@
     if e.__class__.__name__ == "DebuggerInitializationError":
         _is_debugpy_available = False
     else:
-        raise e
+        raise
 
 
 # Required for backwards compatibility
@@ -466,7 +466,7 @@ async def dumpCell(self, message):
         code = message["arguments"]["code"]
         file_name = get_file_name(code)
 
-        with open(file_name, "w", encoding="utf-8") as f:
+        with open(file_name, "w", encoding="utf-8") as f:  # noqa: ASYNC230
             f.write(code)
 
         return {
@@ -496,7 +496,7 @@ async def source(self, message):
         reply = {"type": "response", "request_seq": message["seq"], "command": message["command"]}
         source_path = message["arguments"]["source"]["path"]
         if Path(source_path).is_file():
-            with open(source_path, encoding="utf-8") as f:
+            with open(source_path, encoding="utf-8") as f:  # noqa: ASYNC230
                 reply["success"] = True
                 reply["body"] = {"content": f.read()}
         else:

ipykernel/inprocess/ipkernel.py

@@ -107,7 +107,7 @@ def _input_request(self, prompt, ident, parent, password=False):
                 frontend.stdin_channel.call_handlers(msg)
                 break
         else:
-            logging.error("No frontend found for raw_input request")
+            logging.error("No frontend found for raw_input request")  # noqa: LOG015
             return ""
 
         # Await a response.

ipykernel/ipkernel.py

@@ -691,7 +691,7 @@ def do_apply(self, content, bufs, msg_id, reply_metadata):
             working.update(ns)
             code = f"{resultname} = {fname}(*{argname},**{kwargname})"
             try:
-                exec(code, shell.user_global_ns, shell.user_ns)
+                exec(code, shell.user_global_ns, shell.user_ns)  # noqa: S102
                 result = working.get(resultname)
             finally:
                 for key in ns:

ipykernel/jsonutil.py

@@ -26,7 +26,7 @@
 
 # holy crap, strptime is not threadsafe.
 # Calling it once at import seems to help.
-datetime.strptime("2000-01-01", "%Y-%m-%d")
+datetime.strptime("2000-01-01", "%Y-%m-%d")  # noqa: DTZ007
 
 # -----------------------------------------------------------------------------
 # Classes and functions

ipykernel/kernelapp.py

@@ -698,7 +698,7 @@ def init_pdb(self):
         With the non-interruptible version, stopping pdb() locks up the kernel in a
         non-recoverable state.
         """
-        import pdb
+        import pdb  # noqa: T100
 
         from IPython.core import debugger
 

ipykernel/kernelbase.py

@@ -860,7 +860,7 @@ async def execute_request(self, stream, ident, parent):
         # clients... This seems to mitigate the problem, but we definitely need
         # to better understand what's going on.
         if self._execute_sleep:
-            time.sleep(self._execute_sleep)
+            time.sleep(self._execute_sleep)  # noqa: ASYNC251
 
         # Send the reply.
         reply_content = json_clean(reply_content)
@@ -1547,8 +1547,8 @@ async def _at_shutdown(self):
         """Actions taken at shutdown by the kernel, called by python's atexit."""
         try:
             await self._progressively_terminate_all_children()
-        except Exception as e:
-            self.log.exception("Exception during subprocesses termination %s", e)
+        except Exception:
+            self.log.exception("Exception during subprocesses termination")
 
         finally:
             if self._shutdown_message is not None and self.session:

ipykernel/trio_runner.py

@@ -35,15 +35,15 @@ def interrupt(self, signum, frame):
             self._cell_cancel_scope.cancel()
         else:
             msg = "Kernel interrupted but no cell is running"
-            raise Exception(msg)
+            raise Exception(msg)  # noqa: TRY002
 
     def run(self):
         """Run the loop."""
         old_sig = signal.signal(signal.SIGINT, self.interrupt)
 
         def log_nursery_exc(exc):
             exc = "\n".join(traceback.format_exception(type(exc), exc, exc.__traceback__))
-            logging.error("An exception occurred in a global nursery task.\n%s", exc)
+            logging.error("An exception occurred in a global nursery task.\n%s", exc)  # noqa: LOG015
 
         async def trio_main():
             """Run the main loop."""

pyproject.toml

@@ -271,7 +271,24 @@ ignore = [
 # PLC1901 `stderr == ""` can be simplified to `not stderr` as an empty string is falsey
 # B018 Found useless expression. Either assign it to a variable or remove it.
 # S603 `subprocess` call: check for execution of untrusted input
-"tests/*" = ["B011", "C408", "T201", "B007", "EM", "PTH", "PLW", "PLC1901", "B018", "S603", "ARG", "RET", "PGH"]
+"tests/*" = [
+  "ARG",
+  "B007",
+  "B011",
+  "B018",
+  "BLE001",
+  "C408",
+  "DTZ001",
+  "EM",
+  "FLY002",
+  "PGH",
+  "PLC1901",
+  "PLW",
+  "PTH",
+  "RET",
+  "S603",
+  "T201",
+]
 "*/__init__.py" = ["F401"]
 
 [tool.interrogate]