Pin jupyter client < 8 (#378)

* Cap jupyter_client < 8
* Update log formatting to include timestamp, add version to startup message
* Drop python 3.6 from CI matrix
* Add pre-commit, ensure newline at EOF

Author: kevin-bates

.github/workflows/tests.yml

@@ -31,8 +31,6 @@ jobs:
     strategy:
       matrix:
         python:
-        - "3.6"
-        - "3.6"
         - "3.7"
         - "3.8"
         - "3.9"

.pre-commit-config.yaml

@@ -0,0 +1,5 @@
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.4.0
+    hooks:
+      - id: end-of-file-fixer

docs/_static/numpy_friendly.css

@@ -2,4 +2,4 @@
 
 table.docutils th.field-name {
     white-space: nowrap;
-}
+}

docs/_static/numpy_friendly.css_t

@@ -2,4 +2,4 @@
 
 table.docutils th.field-name {
     white-space: nowrap;
-}
+}

docs/source/index.rst

@@ -81,4 +81,3 @@ Indices and tables
 * :ref:`genindex`
 * :ref:`modindex`
 * :ref:`search`
-

docs/source/troubleshooting.md

@@ -12,4 +12,4 @@ you can access your kernel gateway on the IP address of your Docker host an port
 
 ## Kernel gateway raises an error when I use `notebook-http` mode.
 
-The `notebook-http` mode publishes a web API defined by annotations and code in a notebook. Make sure you are specifying a path or URL of a notebook (`*.ipynb`) file when you launch the kernel gateway in this mode. Set the `--KernelGatewayApp.seed_uri` command line parameter or `KG_SEED_URI` environment variable to do so.
+The `notebook-http` mode publishes a web API defined by annotations and code in a notebook. Make sure you are specifying a path or URL of a notebook (`*.ipynb`) file when you launch the kernel gateway in this mode. Set the `--KernelGatewayApp.seed_uri` command line parameter or `KG_SEED_URI` environment variable to do so.

kernel_gateway/gatewayapp.py

@@ -32,7 +32,7 @@
 
 from tornado import httpserver
 from tornado import web
-from tornado.log import enable_pretty_logging
+from tornado.log import enable_pretty_logging, LogFormatter
 
 from notebook.notebookapp import random_ports
 from ._version import __version__
@@ -310,6 +310,15 @@ def ssl_version_default(self):
         ssl_from_env = os.getenv(self.ssl_version_env)
         return ssl_from_env if ssl_from_env is None else int(ssl_from_env)
 
+    _log_formatter_cls = LogFormatter  # traitlet default is LevelFormatter
+
+    @default("log_format")
+    def _default_log_format(self) -> str:
+        """override default log format to include milliseconds"""
+        return (
+            "%(color)s[%(levelname)1.1s %(asctime)s.%(msecs).03d %(name)s]%(end_color)s %(message)s"
+        )
+
     kernel_spec_manager = Instance(KernelSpecManager, allow_none=True)
 
     kernel_spec_manager_class = Type(
@@ -556,8 +565,8 @@ def init_http_server(self):
     def start(self):
         """Starts an IO loop for the application."""
         super(KernelGatewayApp, self).start()
-        self.log.info('Jupyter Kernel Gateway at http{}://{}:{}'.format(
-            's' if self.keyfile else '', self.ip, self.port
+        self.log.info('Jupyter Kernel Gateway {} is available at http{}://{}:{}'.format(
+            KernelGatewayApp.version, 's' if self.keyfile else '', self.ip, self.port
         ))
         self.io_loop = ioloop.IOLoop.current()
 

kernel_gateway/jupyter_websocket/swagger.json

@@ -590,4 +590,4 @@
       }
     }
   }
-}
+}

kernel_gateway/tests/test_jupyter_websocket.py

@@ -823,4 +823,3 @@ def test_seed_language_support(self):
         self.assertIn('Gur Mra bs Clguba', content['text'])
 
         ws.close()
-

readthedocs.yml

@@ -2,4 +2,4 @@ conda:
     file: docs/environment.yml
 python:
     version: 3
-  
+