removing commented out code

sdk/containerregistry/azure-containerregistry/azure/containerregistry/_base_client.py

@@ -23,7 +23,7 @@ class ContainerRegistryApiVersion(str, Enum):
 
 
 class ContainerRegistryBaseClient(object):
-    """Base class for ContainerRegistryClient and ContainerRepository
+    """Base class for ContainerRegistryClient, ContainerRepository, and RegistryArtifact
 
     :param str endpoint: Azure Container Registry endpoint
     :param credential: AAD Token for authenticating requests with Azure

sdk/containerregistry/azure-containerregistry/azure/containerregistry/_container_repository.py

@@ -52,11 +52,6 @@ def __init__(self, endpoint, repository, credential, **kwargs):
         self._credential = credential
         super(ContainerRepository, self).__init__(endpoint=self._endpoint, credential=credential, **kwargs)
 
-    # def _get_digest_from_tag(self, tag):
-    #     # type: (str) -> str
-    #     tag_props = self.get_tag_properties(tag)
-    #     return tag_props.digest
-
     @distributed_trace
     def delete(self, **kwargs):
         # type: (Dict[str, Any]) -> None
@@ -70,29 +65,6 @@ def delete(self, **kwargs):
             self._client.container_registry.delete_repository(self.repository, **kwargs)
         )
 
-    # @distributed_trace
-    # def delete_registry_artifact(self, digest, **kwargs):
-    #     # type: (str, Dict[str, Any]) -> None
-    #     """Delete a registry artifact
-
-    #     :param digest: The digest of the artifact to be deleted
-    #     :type digest: str
-    #     :returns: None
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     self._client.container_registry.delete_manifest(self.repository, digest, **kwargs)
-
-    # @distributed_trace
-    # def delete_tag(self, tag, **kwargs):
-    #     # type: (str, Dict[str, Any]) -> None
-    #     """Delete a tag from a repository
-
-    #     :param str tag: The tag to be deleted
-    #     :returns: None
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     self._client.container_registry.delete_tag(self.repository, tag, **kwargs)
-
     @distributed_trace
     def get_properties(self, **kwargs):
         # type: (Dict[str, Any]) -> RepositoryProperties
@@ -105,39 +77,6 @@ def get_properties(self, **kwargs):
             self._client.container_registry.get_properties(self.repository, **kwargs)
         )
 
-    # @distributed_trace
-    # def get_registry_artifact_properties(self, tag_or_digest, **kwargs):
-    #     # type: (str, Dict[str, Any]) -> ArtifactManifestProperties
-    #     """Get the properties of a registry artifact
-
-    #     :param tag_or_digest: The tag/digest of a registry artifact
-    #     :type tag_or_digest: str
-    #     :returns: :class:`~azure.containerregistry.ArtifactManifestProperties`
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     if _is_tag(tag_or_digest):
-    #         tag_or_digest = self._get_digest_from_tag(tag_or_digest)
-
-    #     return ArtifactManifestProperties._from_generated(  # pylint: disable=protected-access
-    #         self._client.container_registry.get_manifest_properties(
-    #             self.repository, tag_or_digest, **kwargs
-    #         )
-    #     )
-
-    # @distributed_trace
-    # def get_tag_properties(self, tag, **kwargs):
-    #     # type: (str, Dict[str, Any]) -> TagProperties
-    #     """Get the properties for a tag
-
-    #     :param tag: The tag to get properties for
-    #     :type tag: str
-    #     :returns: :class:`~azure.containerregistry.TagProperties`
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     return TagProperties._from_generated(  # pylint: disable=protected-access
-    #         self._client.container_registry.get_tag_properties(self.repository, tag, **kwargs)
-    #     )
-
     @distributed_trace
     def list_registry_artifacts(self, **kwargs):
         # type: (Dict[str, Any]) -> ItemPaged[ArtifactManifestProperties]
@@ -256,160 +195,6 @@ def get_next(next_link=None):
 
         return ItemPaged(get_next, extract_data)
 
-    # @distributed_trace
-    # def list_tags(self, **kwargs):
-    #     # type: (Dict[str, Any]) -> ItemPaged[TagProperties]
-    #     """List the tags for a repository
-
-    #     :keyword last: Query parameter for the last item in the previous call. Ensuing
-    #         call will return values after last lexically
-    #     :paramtype last: str
-    #     :keyword order_by: Query parameter for ordering by time ascending or descending
-    #     :paramtype order_by: :class:`~azure.containerregistry.TagOrderBy`
-    #     :keyword results_per_page: Number of repositories to return per page
-    #     :paramtype results_per_page: int
-    #     :return: ItemPaged[:class:`~azure.containerregistry.TagProperties`]
-    #     :rtype: :class:`~azure.core.paging.ItemPaged`
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     name = self.repository
-    #     last = kwargs.pop("last", None)
-    #     n = kwargs.pop("results_per_page", None)
-    #     orderby = kwargs.pop("order_by", None)
-    #     digest = kwargs.pop("digest", None)
-    #     cls = kwargs.pop(
-    #         "cls", lambda objs: [TagProperties._from_generated(o) for o in objs]  # pylint: disable=protected-access
-    #     )
-
-    #     error_map = {401: ClientAuthenticationError, 404: ResourceNotFoundError, 409: ResourceExistsError}
-    #     error_map.update(kwargs.pop("error_map", {}))
-    #     accept = "application/json"
-
-    #     def prepare_request(next_link=None):
-    #         # Construct headers
-    #         header_parameters = {}  # type: Dict[str, Any]
-    #         header_parameters["Accept"] = self._client._serialize.header(  # pylint: disable=protected-access
-    #             "accept", accept, "str"
-    #         )
-
-    #         if not next_link:
-    #             # Construct URL
-    #             url = "/acr/v1/{name}/_tags"
-    #             path_format_arguments = {
-    #                 "url": self._client._serialize.url(  # pylint: disable=protected-access
-    #                     "self._config.url",
-    #                     self._client._config.url,  # pylint: disable=protected-access
-    #                     "str",
-    #                     skip_quote=True,
-    #                 ),
-    #                 "name": self._client._serialize.url("name", name, "str"),  # pylint: disable=protected-access
-    #             }
-    #             url = self._client._client.format_url(url, **path_format_arguments)  # pylint: disable=protected-access
-    #             # Construct parameters
-    #             query_parameters = {}  # type: Dict[str, Any]
-    #             if last is not None:
-    #                 query_parameters["last"] = self._client._serialize.query(  # pylint: disable=protected-access
-    #                     "last", last, "str"
-    #                 )
-    #             if n is not None:
-    #                 query_parameters["n"] = self._client._serialize.query(  # pylint: disable=protected-access
-    #                     "n", n, "int"
-    #                 )
-    #             if orderby is not None:
-    #                 query_parameters["orderby"] = self._client._serialize.query(  # pylint: disable=protected-access
-    #                     "orderby", orderby, "str"
-    #                 )
-    #             if digest is not None:
-    #                 query_parameters["digest"] = self._client._serialize.query(  # pylint: disable=protected-access
-    #                     "digest", digest, "str"
-    #                 )
-
-    #             request = self._client._client.get(  # pylint: disable=protected-access
-    #                 url, query_parameters, header_parameters
-    #             )
-    #         else:
-    #             url = next_link
-    #             query_parameters = {}  # type: Dict[str, Any]
-    #             path_format_arguments = {
-    #                 "url": self._client._serialize.url(  # pylint: disable=protected-access
-    #                     "self._client._config.url",
-    #                     self._client._config.url,  # pylint: disable=protected-access
-    #                     "str",
-    #                     skip_quote=True,
-    #                 ),
-    #                 "name": self._client._serialize.url("name", name, "str"),  # pylint: disable=protected-access
-    #             }
-    #             url = self._client._client.format_url(url, **path_format_arguments)  # pylint: disable=protected-access
-    #             request = self._client._client.get(  # pylint: disable=protected-access
-    #                 url, query_parameters, header_parameters
-    #             )
-    #         return request
-
-    #     def extract_data(pipeline_response):
-    #         deserialized = self._client._deserialize("TagList", pipeline_response)  # pylint: disable=protected-access
-    #         list_of_elem = deserialized.tag_attribute_bases
-    #         if cls:
-    #             list_of_elem = cls(list_of_elem)
-    #         link = None
-    #         if "Link" in pipeline_response.http_response.headers.keys():
-    #             link = _parse_next_link(pipeline_response.http_response.headers["Link"])
-    #         return link, iter(list_of_elem)
-
-    #     def get_next(next_link=None):
-    #         request = prepare_request(next_link)
-
-    #         pipeline_response = self._client._client._pipeline.run(  # pylint: disable=protected-access
-    #             request, stream=False, **kwargs
-    #         )
-    #         response = pipeline_response.http_response
-
-    #         if response.status_code not in [200]:
-    #             error = self._client._deserialize.failsafe_deserialize(  # pylint: disable=protected-access
-    #                 AcrErrors, response
-    #             )
-    #             map_error(status_code=response.status_code, response=response, error_map=error_map)
-    #             raise HttpResponseError(response=response, model=error)
-
-    #         return pipeline_response
-
-    #     return ItemPaged(get_next, extract_data)
-
-    # @distributed_trace
-    # def set_manifest_properties(self, digest, permissions, **kwargs):
-    #     # type: (str, ContentPermissions, Dict[str, Any]) -> ArtifactManifestProperties
-    #     """Set the properties for a manifest
-
-    #     :param digest: Digest of a manifest
-    #     :type digest: str
-    #     :param permissions: The property's values to be set
-    #     :type permissions: ContentPermissions
-    #     :returns: :class:`~azure.containerregistry.ArtifactManifestProperties`
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     return ArtifactManifestProperties._from_generated(  # pylint: disable=protected-access
-    #         self._client.container_registry.update_manifest_properties(
-    #             self.repository, digest, value=permissions._to_generated(), **kwargs  # pylint: disable=protected-access
-    #         )
-    #     )
-
-    # @distributed_trace
-    # def set_tag_properties(self, tag, permissions, **kwargs):
-    #     # type: (str, ContentPermissions, Dict[str, Any]) -> TagProperties
-    #     """Set the properties for a tag
-
-    #     :param tag: Tag to set properties for
-    #     :type tag: str
-    #     :param permissions: The property's values to be set
-    #     :type permissions: ContentPermissions
-    #     :returns: :class:`~azure.containerregistry.TagProperties`
-    #     :raises: :class:`~azure.core.exceptions.ResourceNotFoundError`
-    #     """
-    #     return TagProperties._from_generated(  # pylint: disable=protected-access
-    #         self._client.container_registry.update_tag_attributes(
-    #             self.repository, tag, value=permissions._to_generated(), **kwargs  # pylint: disable=protected-access
-    #         )
-    #     )
-
     @distributed_trace
     def set_properties(self, properties, **kwargs):
         # type: (RepositoryProperties, Dict[str, Any]) -> RepositoryProperties

sdk/containerregistry/azure-containerregistry/azure/containerregistry/aio/_async_base_client.py

@@ -22,7 +22,7 @@ class ContainerRegistryApiVersion(str, Enum):
 
 
 class ContainerRegistryBaseClient(object):
-    """Base class for ContainerRegistryClient and ContainerRepository
+    """Base class for ContainerRegistryClient, ContainerRepository, and RegistryArtifact
 
     :param endpoint: Azure Container Registry endpoint
     :type endpoint: str

sdk/containerregistry/azure-containerregistry/azure/containerregistry/aio/_async_container_registry_client.py

@@ -181,8 +181,7 @@ def get_repository_client(self, repository: str, **kwargs: Dict[str, Any]) -> Co
         )
 
     @distributed_trace
-    def get_artifact(self, repository_name, tag_or_digest, **kwargs):
-        # type: (str, str, Dict[str, Any]) -> RegistryArtifact
+    def get_artifact(self, repository_name: str, tag_or_digest: str, **kwargs: Dict[str, Any]) -> RegistryArtifact:
         """Get a Registry Artifact object
 
         :param str repository_name: Name of the repository