Add more ci checks (#1063)

* add more ci checks

* fixes

* lint

* lint

Author: blink1073

.github/workflows/ci.yml

@@ -76,7 +76,7 @@ jobs:
          run: |
           hatch run typing:test
           hatch run lint:style
-          pipx run 'validate-pyproject[all]' pyproject.toml
+          pipx run interrogate -vv .
           pipx run doc8 --max-line-length=200
 
   check_release:
@@ -89,7 +89,7 @@ jobs:
           token: ${{ secrets.GITHUB_TOKEN }}
 
   test_docs:
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     steps:
       - uses: actions/checkout@v3
       - uses: jupyterlab/maintainer-tools/.github/actions/base-setup@v1

.pre-commit-config.yaml

@@ -5,16 +5,19 @@ repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.4.0
     hooks:
-      - id: end-of-file-fixer
       - id: check-case-conflict
+      - id: check-ast
+      - id: check-docstring-first
       - id: check-executables-have-shebangs
-      - id: requirements-txt-fixer
       - id: check-added-large-files
       - id: check-case-conflict
+      - id: check-merge-conflict
+      - id: check-json
       - id: check-toml
       - id: check-yaml
-      - id: forbid-new-submodules
-      - id: check-builtin-literals
+      - id: debug-statements
+        exclude: ipykernel/kernelapp.py
+      - id: end-of-file-fixer
       - id: trailing-whitespace
 
   - repo: https://github.com/python-jsonschema/check-jsonschema
@@ -33,7 +36,7 @@ repos:
       - id: black
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.177
+    rev: v0.0.189
     hooks:
       - id: ruff
         args: ["--fix"]

docs/conf.py

@@ -36,6 +36,14 @@
     "sphinxcontrib_github_alt",
 ]
 
+try:
+    import enchant  # noqa
+
+    extensions += ["sphinxcontrib.spelling"]
+except ImportError:
+    pass
+
+
 github_project_url = "https://github.com/ipython/ipykernel"
 
 # Add any paths that contain templates here, relative to this directory.

examples/embedding/inprocess_qtconsole.py

@@ -1,3 +1,4 @@
+"""An in-process qt console app."""
 import os
 import sys
 
@@ -8,6 +9,7 @@
 
 
 def print_process_id():
+    """Print the process id."""
     print("Process ID is:", os.getpid())
 
 
@@ -42,6 +44,7 @@ def init_asyncio_patch():
 
 
 def main():
+    """The main entry point."""
     # Print the ID of the main process
     print_process_id()
 

examples/embedding/inprocess_terminal.py

@@ -1,3 +1,4 @@
+"""An in-process terminal example."""
 import os
 import sys
 
@@ -8,6 +9,7 @@
 
 
 def print_process_id():
+    """Print the process id."""
     print("Process ID is:", os.getpid())
 
 
@@ -42,6 +44,7 @@ def init_asyncio_patch():
 
 
 def main():
+    """The main function."""
     print_process_id()
 
     # Create an in-process kernel

examples/embedding/internal_ipkernel.py

@@ -1,3 +1,4 @@
+"""An internal ipykernel example."""
 # -----------------------------------------------------------------------------
 # Imports
 # -----------------------------------------------------------------------------
@@ -26,8 +27,10 @@ def mpl_kernel(gui):
 
 
 class InternalIPKernel:
+    """An internal ipykernel class."""
+
     def init_ipkernel(self, backend):
-        # Start IPython kernel with GUI event loop and mpl support
+        """Start IPython kernel with GUI event loop and mpl support."""
         self.ipkernel = mpl_kernel(backend)
         # To create and track active qt consoles
         self.consoles = []
@@ -41,6 +44,7 @@ def init_ipkernel(self, backend):
         # self.namespace['ipkernel'] = self.ipkernel  # dbg
 
     def print_namespace(self, evt=None):
+        """Print the namespace."""
         print("\n***Variables in User namespace***")
         for k, v in self.namespace.items():
             if not k.startswith("_"):
@@ -52,8 +56,10 @@ def new_qt_console(self, evt=None):
         return connect_qtconsole(self.ipkernel.abs_connection_file, profile=self.ipkernel.profile)
 
     def count(self, evt=None):
+        """Get the app counter value."""
         self.namespace["app_counter"] += 1
 
     def cleanup_consoles(self, evt=None):
+        """Clean up the consoles."""
         for c in self.consoles:
             c.kill()

examples/embedding/ipkernel_qtapp.py

@@ -26,13 +26,17 @@
 # Functions and classes
 # -----------------------------------------------------------------------------
 class SimpleWindow(Qt.QWidget, InternalIPKernel):
+    """A custom Qt widget for IPykernel."""
+
     def __init__(self, app):
+        """Initialize the widget."""
         Qt.QWidget.__init__(self)
         self.app = app
         self.add_widgets()
         self.init_ipkernel("qt")
 
     def add_widgets(self):
+        """Add the widget."""
         self.setGeometry(300, 300, 400, 70)
         self.setWindowTitle("IPython in your app")
 

examples/embedding/ipkernel_wxapp.py

@@ -36,6 +36,7 @@ class MyFrame(wx.Frame, InternalIPKernel):
     """
 
     def __init__(self, parent, title):
+        """Initialize the frame."""
         wx.Frame.__init__(self, parent, -1, title, pos=(150, 150), size=(350, 285))
 
         # Create the menubar
@@ -99,7 +100,10 @@ def OnTimeToClose(self, evt):
 
 
 class MyApp(wx.App):
+    """A custom wx app."""
+
     def OnInit(self):
+        """Initialize app."""
         frame = MyFrame(None, "Simple wxPython App")
         self.SetTopWindow(frame)
         frame.Show(True)

hatch_build.py

@@ -1,3 +1,4 @@
+"""A custom hatch build hook for ipykernel."""
 import os
 import shutil
 import sys
@@ -6,7 +7,10 @@
 
 
 class CustomHook(BuildHookInterface):
+    """The IPykernel build hook."""
+
     def initialize(self, version, build_data):
+        """Initialize the hook."""
         here = os.path.abspath(os.path.dirname(__file__))
         sys.path.insert(0, here)
         from ipykernel.kernelspec import make_ipkernel_cmd, write_kernel_spec

ipykernel/__main__.py

@@ -1,3 +1,4 @@
+"""The cli entry point for ipykernel."""
 if __name__ == "__main__":
     from ipykernel import kernelapp as app