Remove Promise around Listen

This seemed like a good initially just because of then/catch, but there are a few problems with this:

1. Calling .listen() does not return the Polka instance directly, so it forces listen to always be called last.

2. It also prevents one-line initializers for a server.

   const server = polka().listen(3000);
   //=> server is Promise, not Polka instance
   

3. The top-level then and catch blocks make it seem like catch is the global catch-all handler.

Instead, .listen() would expose the underlying server.listen method directly, meaning that signature is the same as all other servers.

polka().listen(PORT, err => {
  if (err) throw err; // or whatever you want to do
  console.log(`> Ready on localhost:${PORT}`);
});

This would also remove 5 lines of code 😆

[Youngestdev]

polka().listen(PORT, err => {
  if (err) throw err; // or whatever you want to do
  console.log(`> Ready on localhost:${PORT}`);
});

That looks cool

[lukeed]

Yup! That’s actually exactly the code I was going to write. I just (still) haven’t decided if this should happen or not. Pros & cons for each side.