[Communication] - Sms - Fixed pylint issues (#36361)

* [Communication] - Sms - Updated project documentation README.md and CHANGELOG.md

* Fixed docstring and method signature on"send" function

* Removed unnecessary overwrites in the function "send"

* Rolled back spell check changes in README.md

* Rolled back spell check changes in README.md

Author: dinazavyr

sdk/communication/azure-communication-sms/CHANGELOG.md

@@ -9,7 +9,7 @@
 ### Bugs Fixed
 
 ### Other Changes
-Python 2.7 is no longer supported. Please use Python version 3.7 or later. For more details, please read our page on [Azure SDK for Python version support policy](https://github.com/Azure/azure-sdk-for-python/wiki/Azure-SDKs-Python-version-support-policy).
+- Python 2.7 is no longer supported. Please use Python version 3.7 or later. For more details, please read our page on [Azure SDK for Python version support policy](https://github.com/Azure/azure-sdk-for-python/wiki/Azure-SDKs-Python-version-support-policy).
 
 ## 1.0.1 (2021-06-08)
 

sdk/communication/azure-communication-sms/azure/communication/sms/_sms_client.py

@@ -4,7 +4,7 @@
 # license information.
 # --------------------------------------------------------------------------
 
-from typing import Union
+from typing import List, Union, Any, Optional
 from uuid import uuid4
 from azure.core.tracing.decorator import distributed_trace
 from azure.communication.sms._generated.models import (
@@ -20,7 +20,8 @@
 from ._shared.utils import parse_connection_str, get_current_utc_time
 from ._version import SDK_MONIKER
 
-class SmsClient(object): # pylint: disable=client-accepts-api-version-keyword
+
+class SmsClient(object):  # pylint: disable=client-accepts-api-version-keyword
     """A client to interact with the AzureCommunicationService Sms gateway.
 
     This client provides operations to send an SMS via a phone number.
@@ -30,17 +31,18 @@ class SmsClient(object): # pylint: disable=client-accepts-api-version-keyword
     :param Union[TokenCredential, AzureKeyCredential] credential:
         The credential we use to authenticate against the service.
     """
+
     def __init__(
-            self, endpoint, # type: str
-            credential, # type: Union[TokenCredential, AzureKeyCredential]
-            **kwargs # type: Any
-        ):
+            self, endpoint,  # type: str
+            credential,  # type: Union[TokenCredential, AzureKeyCredential]
+            **kwargs  # type: Any
+    ):
         # type: (...) -> None
         try:
             if not endpoint.lower().startswith('http'):
                 endpoint = "https://" + endpoint
         except AttributeError:
-            raise ValueError("Account URL must be a string.") # pylint: disable=raise-missing-from
+            raise ValueError("Account URL must be a string.")  # pylint: disable=raise-missing-from
 
         if not credential:
             raise ValueError(
@@ -56,8 +58,8 @@ def __init__(
 
     @classmethod
     def from_connection_string(cls, conn_str,  # type: str
-            **kwargs  # type: Any
-        ):  # type: (...) -> SmsClient
+                               **kwargs  # type: Any
+                               ):  # type: (...) -> SmsClient
         """Create SmsClient from a Connection String.
 
         :param str conn_str:
@@ -79,17 +81,20 @@ def from_connection_string(cls, conn_str,  # type: str
         return cls(endpoint, access_key, **kwargs)
 
     @distributed_trace
-    def send(self, from_, # type: str
-             to, # type: Union[str, List[str]]
-             message, # type: str
-             **kwargs #type: Any
-        ): # type: (...) -> [SmsSendResult]
+    def send(self, from_,  # type: str
+             to,  # type: Union[str, List[str]]
+             message,  # type: str,
+             *,
+             enable_delivery_report: bool = False,
+             tag: Optional[str] = None,
+             **kwargs: Any
+             ):  # type: (...) -> [SmsSendResult]
         """Sends SMSs to phone numbers.
 
         :param str from_: The sender of the SMS.
         :param to: The single recipient or the list of recipients of the SMS.
         :type to: Union[str, List[str]]
-        :param str message: The message in the SMS
+        :param str message: The message in the SMS.
         :keyword bool enable_delivery_report: Enable this flag to receive a delivery report for this
          message on the Azure Resource EventGrid.
         :keyword str tag: Use this field to provide metadata that will then be sent back in the corresponding
@@ -101,9 +106,6 @@ def send(self, from_, # type: str
         if isinstance(to, str):
             to = [to]
 
-        enable_delivery_report = kwargs.pop('enable_delivery_report', False)
-        tag = kwargs.pop('tag', None)
-
         sms_send_options = SmsSendOptions(
             enable_delivery_report=enable_delivery_report,
             tag=tag