ms-toolsai.jupyter is now a dependency of ms-python and is installed
along with it.

See @sreya's review

See previous commit for failure introduced.

We should always send HTML if the user agent expects it.

If they do not, they should clearly indicate such via the Accept header.

Closes #2297

Move helm from root and fix stuff in README

Not sure where --name came from? Maybe an older version of helm.

Ah, it's from v2.16.7

Update Kubernetes Helm Chart

I'm not sure why other builds are passing with this still in.

Feels like it parallels better with the other handlers.

Also switch `once` to `on` since we `off` them later anyway so no point
in making Node do it twice.

This will make the route more robust since it'll work under more than
just the root.

Turns out that while Typescript can't infer the callback return type
from it, Typescript can do the opposite and infer it from the callback
return type.

Also add a check that the domain has a dot. This covers the localhost
case as well, so remove that.

Hopefully is a bit easier to read.

That's the default so it's extra visual noise.

Plugin API to add more applications to code-server

Unfortunately we can't use node-mocks-http to test a express.Router
that has async routes. See https://github.com/howardabrams/node-mocks-http/issues/225

router will just return undefined if the executing handler is async and
so the test will have no way to wait for it to complete. Thus, we have
to use supertest which starts an actual HTTP server in the background
and uses a HTTP client to send requests.