Incorporates feedback from pull request

- Adds a few typehints that were missing

Author: weierophinney

src/Application.php

@@ -109,7 +109,7 @@ public function __construct(
      * @param callable|string $middleware Middleware (or middleware service name) to associate with route.
      * @param null|string $name The name of the route.
      */
-    public function get($path, $middleware, string $name = null) : Router\Route
+    public function get(string $path, $middleware, string $name = null) : Router\Route
     {
         return $this->route($path, $middleware, ['GET'], $name);
     }
@@ -118,7 +118,7 @@ public function get($path, $middleware, string $name = null) : Router\Route
      * @param callable|string $middleware Middleware (or middleware service name) to associate with route.
      * @param null|string $name The name of the route.
      */
-    public function post($path, $middleware, $name = null) : Router\Route
+    public function post(string $path, $middleware, $name = null) : Router\Route
     {
         return $this->route($path, $middleware, ['POST'], $name);
     }
@@ -127,7 +127,7 @@ public function post($path, $middleware, $name = null) : Router\Route
      * @param callable|string $middleware Middleware (or middleware service name) to associate with route.
      * @param null|string $name The name of the route.
      */
-    public function put($path, $middleware, string $name = null) : Router\Route
+    public function put(string $path, $middleware, string $name = null) : Router\Route
     {
         return $this->route($path, $middleware, ['PUT'], $name);
     }
@@ -136,7 +136,7 @@ public function put($path, $middleware, string $name = null) : Router\Route
      * @param callable|string $middleware Middleware (or middleware service name) to associate with route.
      * @param null|string $name The name of the route.
      */
-    public function patch($path, $middleware, string $name = null) : Router\Route
+    public function patch(string $path, $middleware, string $name = null) : Router\Route
     {
         return $this->route($path, $middleware, ['PATCH'], $name);
     }
@@ -145,7 +145,7 @@ public function patch($path, $middleware, string $name = null) : Router\Route
      * @param callable|string $middleware Middleware (or middleware service name) to associate with route.
      * @param null|string $name The name of the route.
      */
-    public function delete($path, $middleware, string $name = null) : Router\Route
+    public function delete(string $path, $middleware, string $name = null) : Router\Route
     {
         return $this->route($path, $middleware, ['DELETE'], $name);
     }
@@ -154,7 +154,7 @@ public function delete($path, $middleware, string $name = null) : Router\Route
      * @param callable|string $middleware Middleware (or middleware service name) to associate with route.
      * @param null|string $name The name of the route.
      */
-    public function any($path, $middleware, string $name = null) : Router\Route
+    public function any(string $path, $middleware, string $name = null) : Router\Route
     {
         return $this->route($path, $middleware, null, $name);
     }
@@ -191,6 +191,7 @@ public function any($path, $middleware, string $name = null) : Router\Route
      *
      * @param string|array|callable $path Either a URI path prefix, or middleware.
      * @param null|string|array|callable $middleware Middleware
+     * @return $this
      */
     public function pipe($path, $middleware = null) : parent
     {
@@ -297,7 +298,7 @@ public function route(string $path, $middleware, array $methods = null, string $
      *
      * @return Router\Route[]
      */
-    public function getRoutes() : iterable
+    public function getRoutes() : array
     {
         return $this->routes;
     }

src/ApplicationConfigInjectionTrait.php

@@ -238,7 +238,7 @@ private function createCollectionMapper() : callable
      * The function is useful to reduce an array of pipeline middleware to a
      * priority queue.
      */
-    private function createPriorityQueueReducer()
+    private function createPriorityQueueReducer() : callable
     {
         // $serial is used to ensure that items of the same priority are enqueued
         // in the order in which they are inserted.