Clarify KeyClientBuilderExtension methods params (#14737)

* Clarify KeyClientBuilderExtension methods params

Fixes #14695

* Resolve PR feedback

Author: heaths

sdk/keyvault/Azure.Security.KeyVault.Keys/CHANGELOG.md

@@ -7,6 +7,10 @@
 - Added `KeyVaultKeyIdentifier` to parse key URIs.
 - Added `LocalCryptographyClient` to do cryptography operations locally using a `JsonWebKey`.
 
+### Changed
+
+- Clarified documentation for `KeyClientBuilderExtensions` methods.
+
 ## 4.1.0 (2020-08-11)
 
 ### Added

sdk/keyvault/Azure.Security.KeyVault.Keys/src/KeyClientBuilderExtensions.cs

@@ -16,6 +16,10 @@ public static class KeyClientBuilderExtensions
         /// <summary>
         /// Registers a <see cref="KeyClient"/> instance with the provided <paramref name="vaultUri"/>
         /// </summary>
+        /// <typeparam name="TBuilder">The type of builder to extend.</typeparam>
+        /// <param name="builder">The builder to extend.</param>
+        /// <param name="vaultUri">The URI to an Azure Key Vault, for example: https://my-vault.vault.azure.net</param>
+        /// <returns>An Azure client builder.</returns>
         public static IAzureClientBuilder<KeyClient, KeyClientOptions> AddKeyClient<TBuilder>(this TBuilder builder, Uri vaultUri)
             where TBuilder : IAzureClientFactoryBuilderWithCredential
         {
@@ -25,6 +29,11 @@ public static IAzureClientBuilder<KeyClient, KeyClientOptions> AddKeyClient<TBui
         /// <summary>
         /// Registers a <see cref="KeyClient"/> instance with connection options loaded from the provided <paramref name="configuration"/> instance.
         /// </summary>
+        /// <typeparam name="TBuilder">The type of builder to extend.</typeparam>
+        /// <typeparam name="TConfiguration">The type of configuration to use for the client builder.</typeparam>
+        /// <param name="builder">The builder to extend.</param>
+        /// <param name="configuration">The configuration to use for the client builder.</param>
+        /// <returns>An Azure client builder.</returns>
         public static IAzureClientBuilder<KeyClient, KeyClientOptions> AddKeyClient<TBuilder, TConfiguration>(this TBuilder builder, TConfiguration configuration)
             where TBuilder : IAzureClientFactoryBuilderWithConfiguration<TConfiguration>
         {
@@ -34,6 +43,10 @@ public static IAzureClientBuilder<KeyClient, KeyClientOptions> AddKeyClient<TBui
         /// <summary>
         /// Registers a <see cref="KeyClient"/> instance with the provided <paramref name="vaultUri"/>
         /// </summary>
+        /// <typeparam name="TBuilder">The type of builder to extend.</typeparam>
+        /// <param name="builder">The builder to extend.</param>
+        /// <param name="vaultUri">The URI to a specific key in an Azure Key Vault, for example: https://my-vault.vault.azure.net/keys/my-key</param>
+        /// <returns>An Azure client builder.</returns>
         public static IAzureClientBuilder<CryptographyClient, CryptographyClientOptions> AddCryptographyClient<TBuilder>(this TBuilder builder, Uri vaultUri)
             where TBuilder : IAzureClientFactoryBuilderWithCredential
         {
@@ -43,6 +56,11 @@ public static IAzureClientBuilder<CryptographyClient, CryptographyClientOptions>
         /// <summary>
         /// Registers a <see cref="KeyClient"/> instance with connection options loaded from the provided <paramref name="configuration"/> instance.
         /// </summary>
+        /// <typeparam name="TBuilder">The type of builder to extend.</typeparam>
+        /// <typeparam name="TConfiguration">The type of configuration to use for the client builder.</typeparam>
+        /// <param name="builder">The builder to extend.</param>
+        /// <param name="configuration">The configuration to use for the client builder.</param>
+        /// <returns>An Azure client builder.</returns>
         public static IAzureClientBuilder<CryptographyClient, CryptographyClientOptions> AddCryptographyClient<TBuilder, TConfiguration>(this TBuilder builder, TConfiguration configuration)
             where TBuilder : IAzureClientFactoryBuilderWithConfiguration<TConfiguration>
         {