v1.1.0 pylint applied.

score: 9.96/10

Author: mshaileshr@gmail.com

contentstack/https_connection.py

@@ -32,13 +32,11 @@ def user_agents():
     header = {'sdk': dict(name=contentstack.__package__, version=contentstack.__version__),
               'os': get_os_platform,
               'Content-Type': 'application/json'}
-    package = "contentstack-python, - {}".format(contentstack.__version__)
+    package = "contentstack-python/v{}".format(contentstack.__version__)
     return {'User-Agent': str(header), "X-User-Agent": package}
 
 
-# R0903: Too few public methods (1/2) (too-few-public-methods)
-# "pylint doesn't know what's best" - use your own judgement but as a rule.
-class HTTPSConnection:
+class HTTPSConnection:  # R0903: Too few public methods
     """Make Https Request to fetch the result as per requested url"""
 
     def __init__(self, endpoint, headers):
@@ -72,15 +70,3 @@ def get(self, url):
     def update_connection_timeout(self, timeout: int):
         """Facilitate to update timeout for the https request"""
         self.default_timeout = timeout
-
-    # def get_complete_url(self, base_url: str, params: dict):
-    #     """
-    #     creates complete url with the help of base_url and query parameters
-    #     :param base_url: base url
-    #     :param params: query parameters
-    #     :return: url
-    #     """
-    #     if 'query' in params:
-    #         params["query"] = json.dumps(params["query"])
-    #     query = urlparse.urlencode(params)
-    #     return '{}&{}'.format(base_url, query)

contentstack/utility.py

@@ -21,7 +21,7 @@ def config_logging(logging_type: logging.WARNING):
     :return: basicConfig instance
     """
     logging.basicConfig(
-        filename='application.log',
+        filename='app.log',
         level=logging_type,
         format='[%(asctime)s] {%(pathname)s:%(lineno)d} %(levelname)s - %(message)s',
         datefmt='%H:%M:%S'
@@ -63,8 +63,7 @@ def do_url_encode(params):
 
     @staticmethod
     def get_complete_url(base_url: str, params: dict):
-        """
-        creates complete url using base_url and their respective parameters
+        """ creates complete url using base_url and their respective parameters
         :param base_url:
         :param params:
         :return:

requirements.txt

@@ -1,4 +1,5 @@
 docutils~=0.16
+py~=1.9.0
 dnspython~=2.0.0
 idna~=2.10
 pip~=20.2.1
@@ -8,6 +9,7 @@ pytz~=2020.1
 Babel~=2.8.0
 pkginfo~=1.5.0.1
 MarkupSafe~=2.0.0a1
+pytest~=6.0.1
 Sphinx~=3.1.2
 packaging~=20.4
 Jinja2~=3.0.0a1
@@ -16,19 +18,15 @@ enum34~=1.1.10
 requests~=2.24.0
 snowballstemmer~=2.0.0
 Pygments~=2.6.1
+attrs~=19.3.0
 certifi~=2020.6.20
 chardet~=3.0.4
+wcwidth~=0.2.5
 six~=1.15.0
+coverage~=5.2.1
 toml~=0.10.1
 urllib3~=1.25.10
 alabaster~=0.7.12
-pyparsing~=3.0.0.a2
+pyparsing~=3.0.0a2
 setuptools~=49.2.1
-zipp~=3.1.0
-html-testRunner~=1.2.1
-py~=1.9.0
-pytest~=6.0.1
-attrs~=19.3.0
-pluggy~=1.0.0.dev0
-wcwidth~=0.2.5
-coverage~=5.2.1
+zipp~=3.1.0

setup.py

@@ -10,11 +10,15 @@
 except ImportError:
     from distutils.core import setup
 
-with open(os.path.join(os.path.dirname(__file__), 'README.rst')) as readme:
+with open(os.path.join(os.path.dirname(__file__), 'readme.rst')) as readme:
     long_description = readme.read()
 
 setup(
-    name="contentstack",
+    title="contentstack-python sdk for content-delivery-api",
+    name="Contentstack",
+    status="Active",
+    type="process",
+    created="17 Jun",
     keywords='contentstack-python',
     version="1.1.0",
     author="Contentstack",