qa: incorporate feedback from review

- Removes unnecessary parameter annotations.
- Adds typehints where needed and useful.
- Fixes annotations where they were incorrect.

src/ConfigPostProcessor.php

@@ -87,7 +87,7 @@ function ($value, array $keys) {
      *     nested configuration.
      * @return array
      */
-    public function __invoke(array $config, $keys = [])
+    public function __invoke(array $config, array $keys = [])
     {
         $rewritten = [];
 
@@ -162,8 +162,6 @@ private function replace($value, array $keys, $key = null)
      * Based on zend-stdlib Zend\Stdlib\ArrayUtils::merge
      * @copyright Copyright (c) 2005-2015 Zend Technologies USA Inc. (http://www.zend.com)
      *
-     * @param  array $a
-     * @param  array $b
      * @return array
      */
     public static function merge(array $a, array $b)
@@ -243,7 +241,6 @@ private function replaceExactValue($value)
      * This same logic can be used for invokables, which are essentially just
      * an alias map.
      *
-     * @param array $value
      * @return array
      */
     private function replaceDependencyAliases(array $aliases)

src/Module.php

@@ -37,7 +37,7 @@ public function init($moduleManager)
      * Type-hinting deliberately omitted to allow unit testing
      * without dependencies on packages that do not exist yet.
      *
-     * @param ModuleEvent $moduleEvent
+     * @param ModuleEvent $event
      */
     public function onMergeConfig($event)
     {