Merge pull request #465 from Shopify/paulinakhew-add-autopep8

Add autopep8

Author: Paulinakhew

.pre-commit-config.yaml

@@ -6,3 +6,7 @@ repos:
     hooks:
     -   id: end-of-file-fixer
     -   id: trailing-whitespace
+-   repo: https://github.com/pre-commit/mirrors-autopep8
+    rev: v1.5.4
+    hooks:
+    -   id: autopep8

pyproject.toml

@@ -0,0 +1,2 @@
+[tool.autopep8]
+max_line_length = 120

scripts/shopify_api.py

@@ -12,23 +12,27 @@
 import six
 from six.moves import input, map
 
+
 def start_interpreter(**variables):
     # add the current working directory to the sys paths
     sys.path.append(os.getcwd())
     console = type('shopify ' + shopify.version.VERSION, (code.InteractiveConsole, object), {})
     import readline
     console(variables).interact()
 
+
 class ConfigFileError(Exception):
     pass
 
+
 def usage(usage_string):
     """Decorator to add a usage string to a function"""
     def decorate(func):
         func.usage = usage_string
         return func
     return decorate
 
+
 class TasksMeta(type):
     _prog = os.path.basename(sys.argv[0])
 
@@ -37,6 +41,7 @@ def __new__(mcs, name, bases, new_attrs):
 
         tasks = list(new_attrs.keys())
         tasks.append("help")
+
         def filter_func(item):
             return not item.startswith("_") and hasattr(getattr(cls, item), "__call__")
         tasks = filter(filter_func, tasks)
@@ -222,7 +227,6 @@ def _default_connection_target(cls):
         target = os.readlink(cls._default_symlink)
         return os.path.join(cls._shop_config_dir, target)
 
-
     @classmethod
     def _default_connection(cls):
         target = cls._default_connection_target()
@@ -253,6 +257,7 @@ def _is_default(cls, connection):
     def _no_config_file_error(cls, filename):
         raise ConfigFileError("There is no config file at " + filename)
 
+
 try:
     Tasks.run_task(*sys.argv[1:])
 except ConfigFileError as e:

setup.py

@@ -1,9 +1,9 @@
 from setuptools import setup
 
-NAME='ShopifyAPI'
+NAME = 'ShopifyAPI'
 exec(open('shopify/version.py').read())
-DESCRIPTION='Shopify API for Python'
-LONG_DESCRIPTION="""\
+DESCRIPTION = 'Shopify API for Python'
+LONG_DESCRIPTION = """\
 The ShopifyAPI library allows python developers to programmatically
 access the admin section of stores using an ActiveResource like
 interface similar the ruby Shopify API gem. The library makes HTTP
@@ -27,7 +27,7 @@
       ],
       test_suite='test',
       tests_require=[
-        'mock>=1.0.1',
+          'mock>=1.0.1',
       ],
       platforms='Any',
       classifiers=['Development Status :: 5 - Production/Stable',

shopify/api_version.py

@@ -2,12 +2,15 @@
 import json
 from six.moves.urllib import request
 
+
 class InvalidVersionError(Exception):
     pass
 
+
 class VersionNotFoundError(Exception):
     pass
 
+
 class ApiVersion(object):
     versions = {}
 
@@ -76,7 +79,7 @@ class Unstable(ApiVersion):
     def __init__(self):
         self._name = 'unstable'
         self._numeric_version = 9000000
-        self._path =  '/admin/api/unstable'
+        self._path = '/admin/api/unstable'
 
     @property
     def stable(self):

shopify/collection.py

@@ -2,6 +2,7 @@
 from six.moves.urllib.parse import urlparse, parse_qs
 import cgi
 
+
 class PaginatedCollection(Collection):
     """
     A subclass of Collection which allows cycling through pages of
@@ -141,6 +142,7 @@ class PaginatedIterator(object):
     ...
     # every page and the page items are iterated
     """
+
     def __init__(self, collection):
         if not isinstance(collection, PaginatedCollection):
             raise TypeError("PaginatedIterator expects a PaginatedCollection instance")

shopify/mixins.py

@@ -1,5 +1,6 @@
 import shopify.resources
 
+
 class Countable(object):
 
     @classmethod

shopify/resources/api_permission.py

@@ -1,5 +1,6 @@
 from ..base import ShopifyResource
 
+
 class ApiPermission(ShopifyResource):
 
     @classmethod

shopify/resources/application_credit.py

@@ -1,4 +1,5 @@
 from ..base import ShopifyResource
 
+
 class ApplicationCredit(ShopifyResource):
     pass

shopify/resources/asset.py

@@ -18,7 +18,7 @@ def _prefix(cls, options={}):
     def _element_path(cls, id, prefix_options={}, query_options=None):
         if query_options is None:
             prefix_options, query_options = cls._split_options(prefix_options)
-        return "%s%s.%s%s" % (cls._prefix(prefix_options)+'/', cls.plural,
+        return "%s%s.%s%s" % (cls._prefix(prefix_options) + '/', cls.plural,
                               cls.format.extension, cls._query_string(query_options))
 
     @classmethod