Address comments

Author: Frank Natividad

storage/src/activate_hmac_key.php

@@ -41,7 +41,7 @@ function activate_hmac_key($accessId, $projectId)
 
     $hmacKey->update('ACTIVE');
 
-    print("The HMAC key is now active." . PHP_EOL);
-    printf("HMAC key Metadata: %s" . PHP_EOL, print_r($hmacKey->info(), true));
+    print('The HMAC key is now active.' . PHP_EOL);
+    printf('HMAC key Metadata: %s' . PHP_EOL, print_r($hmacKey->info(), true));
 }
 # [END storage_activate_hmac_key]

storage/src/create_hmac_key.php

@@ -39,8 +39,8 @@ function create_hmac_key($serviceAccountEmail, $projectId)
     // By default createHmacKey will use the projectId used by StorageClient().
     $hmacKeyCreated = $storage->createHmacKey($serviceAccountEmail, ['projectId' => $projectId]);
 
-    printf("The base64 encoded secret is: %s" . PHP_EOL, $hmacKeyCreated->secret());
-    print("Do not miss that secret, there is no API to recover it." . PHP_EOL);
-    printf("HMAC key Metadata: %s" . PHP_EOL, print_r($hmacKeyCreated->hmacKey()->info(), true));
+    printf('The base64 encoded secret is: %s' . PHP_EOL, $hmacKeyCreated->secret());
+    print('Do not miss that secret, there is no API to recover it.' . PHP_EOL);
+    printf('HMAC key Metadata: %s' . PHP_EOL, print_r($hmacKeyCreated->hmacKey()->info(), true));
 }
 # [END storage_create_hmac_key]

storage/src/deactivate_hmac_key.php

@@ -41,7 +41,7 @@ function deactivate_hmac_key($accessId, $projectId)
 
     $hmacKey->update('INACTIVE');
 
-    print("The HMAC key is now inactive." . PHP_EOL);
-    printf("HMAC key Metadata: %s" . PHP_EOL, print_r($hmacKey->info(), true));
+    print('The HMAC key is now inactive.' . PHP_EOL);
+    printf('HMAC key Metadata: %s' . PHP_EOL, print_r($hmacKey->info(), true));
 }
 # [END storage_deactivate_hmac_key]

storage/src/delete_hmac_key.php

@@ -40,7 +40,9 @@ function delete_hmac_key($accessId, $projectId)
     $hmacKey = $storage->hmacKey($accessId, $projectId);
 
     $hmacKey->delete();
-    print("The key is deleted, though it may still appear in results StorageClient.hmacKeys(\$options=['showDeletedKeys'=>true])");
-    print(" when is used." . PHP_EOL);
+    print(
+      'The key is deleted, though it may still appear in the results of calls ' .
+      'to StorageClient.hmacKeys([\'showDeletedKeys\' => true])' . PHP_EOL
+    );
 }
 # [END storage_get_hmac_key]

storage/src/get_hmac_key.php

@@ -38,6 +38,6 @@ function get_hmac_key($accessId, $projectId)
     $storage = new StorageClient();
     $hmacKey = $storage->hmacKey($accessId, $projectId);
 
-    printf("HMAC key Metadata: %s" . PHP_EOL, print_r($hmacKey->info(), true));
+    printf('HMAC key Metadata: %s' . PHP_EOL, print_r($hmacKey->info(), true));
 }
 # [END storage_get_hmac_key]

storage/src/list_hmac_keys.php

@@ -39,7 +39,7 @@ function list_hmac_keys($projectId)
     $hmacKeys = $storage->hmacKeys(['projectId' => $projectId]);
 
     foreach ($hmacKeys as $hmacKey) {
-        printf("HMAC key Metadata: %s" . PHP_EOL, print_r($hmacKey->info(), true));
+        printf('HMAC key Metadata: %s' . PHP_EOL, print_r($hmacKey->info(), true));
     }
 }
 # [END storage_list_hmac_keys]

storage/storage.php

@@ -490,7 +490,9 @@
         } elseif ($input->getOption('delete')) {
             delete_hmac_key($accessId, $projectId);
         } else {
-            throw new \Exception('You must provide --activate, --deactivate, --get, or --delete with an HMAC key accessId.');
+          throw new \Exception(
+            'You must provide --activate, --deactivate, --get, or --delete with an HMAC key accessId.'
+          );
         }
     });