Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pylint] Suppressing pylint for ACR, Tables #24002

Merged
merged 2 commits into from
Apr 14, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
from azure.core.credentials import TokenCredential


class AnonymousACRExchangeClient(object):
class AnonymousACRExchangeClient(object): # pylint: disable=client-accepts-api-version-keyword
"""Class for handling oauth authentication requests

:param endpoint: Azure Container Registry endpoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,14 @@
from azure.core.credentials import TokenCredential


class ContainerRegistryApiVersion(str, Enum):

class ContainerRegistryApiVersion(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
"""Container Registry API version supported by this package"""

V0_PREVIEW = ""


class ContainerRegistryBaseClient(object):
class ContainerRegistryBaseClient(object): # pylint: disable=client-accepts-api-version-keyword
"""Base class for ContainerRegistryClient

:param str endpoint: Azure Container Registry endpoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ def on_response(self, request, response):
pass


class ACRExchangeClient(object):
class ACRExchangeClient(object): # pylint: disable=client-accepts-api-version-keyword
"""Class for handling oauth authentication requests

:param endpoint: Azure Container Registry endpoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -312,7 +312,7 @@ def repository_name(self):
return self._repository_name


class ArtifactArchitecture(str, Enum):
class ArtifactArchitecture(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta

AMD64 = "amd64"
ARM = "arm"
Expand All @@ -329,7 +329,7 @@ class ArtifactArchitecture(str, Enum):
WASM = "wasm"


class ArtifactOperatingSystem(str, Enum):
class ArtifactOperatingSystem(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta

AIX = "aix"
ANDROID = "android"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
from azure.core.credentials_async import AsyncTokenCredential


class AnonymousACRExchangeClient(object):
class AnonymousACRExchangeClient(object): # pylint: disable=client-accepts-api-version-keyword
"""Class for handling oauth authentication requests

:param endpoint: Azure Container Registry endpoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,13 @@
from azure.core.credentials_async import AsyncTokenCredential


class ContainerRegistryApiVersion(str, Enum):
class ContainerRegistryApiVersion(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
"""Container Registry API version supported by this package"""

V0_PREVIEW = ""


class ContainerRegistryBaseClient(object):
class ContainerRegistryBaseClient(object): # pylint: disable=client-accepts-api-version-keyword
"""Base class for ContainerRegistryClient

:param endpoint: Azure Container Registry endpoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ def on_response(self, request: PipelineRequest, response: PipelineResponse) -> N
pass


class ACRExchangeClient(object):
class ACRExchangeClient(object): # pylint: disable=client-accepts-api-version-keyword
"""Class for handling oauth authentication requests

:param endpoint: Azure Container Registry endpoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ def api_version(self):
return self._client._config.version # pylint: disable=protected-access


class TablesBaseClient(AccountHostsMixin):
class TablesBaseClient(AccountHostsMixin): # pylint: disable=client-accepts-api-version-keyword

def __init__( # pylint: disable=missing-client-constructor-parameter-credential
self,
Expand Down
2 changes: 1 addition & 1 deletion sdk/tables/azure-data-tables/azure/data/tables/_entity.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ def metadata(self):
return self._metadata


class EdmType(str, Enum):
class EdmType(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
"""
Used by :class:`~.EntityProperty` to represent the type of the entity property
to be stored by the Table service.
Expand Down
4 changes: 2 additions & 2 deletions sdk/tables/azure-data-tables/azure/data/tables/_error.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,8 +236,8 @@ def _extract_index(self):
class RequestTooLargeError(TableTransactionError):
"""An error response with status code 413 - Request Entity Too Large"""


class TableErrorCode(str, Enum):
# pylint: disable=enum-must-be-uppercase
class TableErrorCode(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
# Generic storage values
account_already_exists = "AccountAlreadyExists"
account_being_created = "AccountBeingCreated"
Expand Down
8 changes: 4 additions & 4 deletions sdk/tables/azure-data-tables/azure/data/tables/_models.py
Original file line number Diff line number Diff line change
Expand Up @@ -535,24 +535,24 @@ class TablePayloadFormat(object):
"""Returns minimal type information for the entity properties plus some extra odata properties."""


class UpdateMode(str, Enum):
class UpdateMode(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
REPLACE = "replace"
MERGE = "merge"


class TransactionOperation(str, Enum):
class TransactionOperation(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
CREATE = "create"
UPSERT = "upsert"
UPDATE = "update"
DELETE = "delete"


class SASProtocol(str, Enum):
class SASProtocol(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
HTTPS = "https"
HTTP = "http"


class LocationMode(str, Enum):
class LocationMode(str, Enum): # pylint: disable=enum-must-inherit-case-insensitive-enum-meta
"""
Specifies the location the request should be sent to. This mode only applies
for RA-GRS accounts which allow secondary read access. All other account types
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
from azure.core.credentials import AzureNamedKeyCredential, AzureSasCredential


class TableClient(TablesBaseClient):
class TableClient(TablesBaseClient): # pylint: disable=client-accepts-api-version-keyword
"""A client to interact with a specific Table in an Azure Tables account.

:ivar str account_name: The name of the Tables account.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
from azure.core.credentials_async import AsyncTokenCredential


class AsyncTablesBaseClient(AccountHostsMixin):
class AsyncTablesBaseClient(AccountHostsMixin): # pylint: disable=client-accepts-api-version-keyword

def __init__( # pylint: disable=missing-client-constructor-parameter-credential
self,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
from azure.core.credentials_async import AsyncTokenCredential


class TableClient(AsyncTablesBaseClient):
class TableClient(AsyncTablesBaseClient): # pylint: disable=client-accepts-api-version-keyword
"""A client to interact with a specific Table in an Azure Tables account.

:ivar str account_name: The name of the Tables account.
Expand Down