fix array vs varray issue (#16)

`varray<string>` would be accurate, but I figured `KeyedContainer` is safer in case someone's using this with a `darray` somehow, or if we decide to migrate to `vec` eventually.

Author: jjergus

src/RouterCLILookupCodegenBuilder.hack

@@ -207,7 +207,7 @@ final class RouterCLILookupCodegenBuilder {
 
   private function getMainMethod(): CodegenMethod {
     return $this->cg->codegenMethod('main')
-      ->addParameter('array<string> $argv')
+      ->addParameter('KeyedContainer<int, string> $argv')
       ->setReturnType('void')
       ->setBody(
         $this->cg->codegenHackBuilder()

tests/examples/codegen/lookup-path.php

@@ -7,7 +7,7 @@
  * To re-generate this file run vendor/hhvm/hacktest/bin/hacktest
  *
  *
- * @partially-generated SignedSource<<dcbaf7d5f4a45c2c3e92a2b822b556d2>>
+ * @partially-generated SignedSource<<584ccd4103e1cb5ddaa5cdc80ba52017>>
  */
 namespace Facebook\HackRouter\CodeGen\Tests\Generated;
 /* BEGIN MANUAL SECTION init */
@@ -67,7 +67,7 @@ private function getControllersForPath(
     }
   }
 
-  public function main(array<string> $argv): void {
+  public function main(KeyedContainer<int, string> $argv): void {
     $path = $argv[1] ?? null;
     if ($path === null) {
       \fprintf(\STDERR, "Usage: %s PATH\n", $argv[0]);