From e97d7f31e10df96e3c9409a5cffa32ecd8a680f3 Mon Sep 17 00:00:00 2001
From: Anne Thompson <annelo@microsoft.com>
Date: Tue, 4 May 2021 09:50:24 -0700
Subject: [PATCH] Standardize on repositoryName parameter name in
 ContainerRegistryClient methods (#20832)

* fix #20831

* update API listing
---
 ...ainers.ContainerRegistry.netstandard2.0.cs |  6 ++--
 .../src/ContainerRegistryClient.cs            | 36 +++++++++----------
 2 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/sdk/containerregistry/Azure.Containers.ContainerRegistry/api/Azure.Containers.ContainerRegistry.netstandard2.0.cs b/sdk/containerregistry/Azure.Containers.ContainerRegistry/api/Azure.Containers.ContainerRegistry.netstandard2.0.cs
index ce13dc0ce09f..d98543655dd7 100644
--- a/sdk/containerregistry/Azure.Containers.ContainerRegistry/api/Azure.Containers.ContainerRegistry.netstandard2.0.cs
+++ b/sdk/containerregistry/Azure.Containers.ContainerRegistry/api/Azure.Containers.ContainerRegistry.netstandard2.0.cs
@@ -91,10 +91,10 @@ public ContainerRegistryClient(System.Uri registryUri, Azure.Core.TokenCredentia
         public virtual string LoginServer { get { throw null; } }
         public virtual string Name { get { throw null; } }
         public virtual System.Uri RegistryUri { get { throw null; } }
-        public virtual Azure.Response<Azure.Containers.ContainerRegistry.DeleteRepositoryResult> DeleteRepository(string repository, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }
-        public virtual System.Threading.Tasks.Task<Azure.Response<Azure.Containers.ContainerRegistry.DeleteRepositoryResult>> DeleteRepositoryAsync(string repository, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }
+        public virtual Azure.Response<Azure.Containers.ContainerRegistry.DeleteRepositoryResult> DeleteRepository(string repositoryName, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }
+        public virtual System.Threading.Tasks.Task<Azure.Response<Azure.Containers.ContainerRegistry.DeleteRepositoryResult>> DeleteRepositoryAsync(string repositoryName, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }
         public virtual Azure.Containers.ContainerRegistry.RegistryArtifact GetArtifact(string repositoryName, string tagOrDigest) { throw null; }
-        public virtual Azure.Containers.ContainerRegistry.ContainerRepository GetRepository(string name) { throw null; }
+        public virtual Azure.Containers.ContainerRegistry.ContainerRepository GetRepository(string repositoryName) { throw null; }
         public virtual Azure.Pageable<string> GetRepositoryNames(System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }
         public virtual Azure.AsyncPageable<string> GetRepositoryNamesAsync(System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }
     }
diff --git a/sdk/containerregistry/Azure.Containers.ContainerRegistry/src/ContainerRegistryClient.cs b/sdk/containerregistry/Azure.Containers.ContainerRegistry/src/ContainerRegistryClient.cs
index 5bbe9287b53d..3f831f81a558 100644
--- a/sdk/containerregistry/Azure.Containers.ContainerRegistry/src/ContainerRegistryClient.cs
+++ b/sdk/containerregistry/Azure.Containers.ContainerRegistry/src/ContainerRegistryClient.cs
@@ -176,20 +176,20 @@ internal static string ParseUriReferenceFromLinkHeader(string linkValue)
         }
 
         /// <summary> Delete the repository identified by `repostitory`. </summary>
-        /// <param name="repository"> Repository name (including the namespace). </param>
+        /// <param name="repositoryName"> Repository name (including the namespace). </param>
         /// <param name="cancellationToken"> The cancellation token to use. </param>
-        /// <exception cref="ArgumentNullException"> Thrown when <paramref name="repository"/> is null. </exception>
-        /// <exception cref="ArgumentException"> Thrown when <paramref name="repository"/> is empty. </exception>
+        /// <exception cref="ArgumentNullException"> Thrown when <paramref name="repositoryName"/> is null. </exception>
+        /// <exception cref="ArgumentException"> Thrown when <paramref name="repositoryName"/> is empty. </exception>
         /// <exception cref="RequestFailedException">Thrown when a failure is returned by the Container Registry service.</exception>
-        public virtual async Task<Response<DeleteRepositoryResult>> DeleteRepositoryAsync(string repository, CancellationToken cancellationToken = default)
+        public virtual async Task<Response<DeleteRepositoryResult>> DeleteRepositoryAsync(string repositoryName, CancellationToken cancellationToken = default)
         {
-            Argument.AssertNotNullOrEmpty(repository, nameof(repository));
+            Argument.AssertNotNullOrEmpty(repositoryName, nameof(repositoryName));
 
             using DiagnosticScope scope = _clientDiagnostics.CreateScope($"{nameof(ContainerRegistryClient)}.{nameof(DeleteRepository)}");
             scope.Start();
             try
             {
-                return await _restClient.DeleteRepositoryAsync(repository, cancellationToken).ConfigureAwait(false);
+                return await _restClient.DeleteRepositoryAsync(repositoryName, cancellationToken).ConfigureAwait(false);
             }
             catch (Exception e)
             {
@@ -199,20 +199,20 @@ public virtual async Task<Response<DeleteRepositoryResult>> DeleteRepositoryAsyn
         }
 
         /// <summary> Delete the repository identified by `repostitory`. </summary>
-        /// <param name="repository"> Repository name (including the namespace). </param>
+        /// <param name="repositoryName"> Repository name (including the namespace). </param>
         /// <param name="cancellationToken"> The cancellation token to use. </param>
-        /// <exception cref="ArgumentNullException"> Thrown when <paramref name="repository"/> is null. </exception>
-        /// <exception cref="ArgumentException"> Thrown when <paramref name="repository"/> is empty. </exception>
+        /// <exception cref="ArgumentNullException"> Thrown when <paramref name="repositoryName"/> is null. </exception>
+        /// <exception cref="ArgumentException"> Thrown when <paramref name="repositoryName"/> is empty. </exception>
         /// <exception cref="RequestFailedException">Thrown when a failure is returned by the Container Registry service.</exception>
-        public virtual Response<DeleteRepositoryResult> DeleteRepository(string repository, CancellationToken cancellationToken = default)
+        public virtual Response<DeleteRepositoryResult> DeleteRepository(string repositoryName, CancellationToken cancellationToken = default)
         {
-            Argument.AssertNotNullOrEmpty(repository, nameof(repository));
+            Argument.AssertNotNullOrEmpty(repositoryName, nameof(repositoryName));
 
             using DiagnosticScope scope = _clientDiagnostics.CreateScope($"{nameof(ContainerRegistryClient)}.{nameof(DeleteRepository)}");
             scope.Start();
             try
             {
-                return _restClient.DeleteRepository(repository, cancellationToken);
+                return _restClient.DeleteRepository(repositoryName, cancellationToken);
             }
             catch (Exception e)
             {
@@ -224,17 +224,17 @@ public virtual Response<DeleteRepositoryResult> DeleteRepository(string reposito
         /// <summary>
         /// Create a new <see cref="ContainerRepository"/> object for the specified repository.
         /// </summary>
-        /// <param name="name"> The name of the repository to reference. </param>
+        /// <param name="repositoryName"> The name of the repository to reference. </param>
         /// <returns> A new <see cref="ContainerRepository"/> for the desired repository. </returns>
-        /// <exception cref="ArgumentNullException"> Thrown when <paramref name="name"/> is null. </exception>
-        /// <exception cref="ArgumentException"> Thrown when <paramref name="name"/> is empty. </exception>
-        public virtual ContainerRepository GetRepository(string name)
+        /// <exception cref="ArgumentNullException"> Thrown when <paramref name="repositoryName"/> is null. </exception>
+        /// <exception cref="ArgumentException"> Thrown when <paramref name="repositoryName"/> is empty. </exception>
+        public virtual ContainerRepository GetRepository(string repositoryName)
         {
-            Argument.AssertNotNullOrEmpty(name, nameof(name));
+            Argument.AssertNotNullOrEmpty(repositoryName, nameof(repositoryName));
 
             return new ContainerRepository(
                 _registryUri,
-                name,
+                repositoryName,
                 _clientDiagnostics,
                 _restClient);
         }