Merge pull request #730 from dan98765/add_pre_commit_tool_to_repo

Add pre-commit tool to repository, and run on all files

Author: syrusakbary

.pre-commit-config.yaml

@@ -0,0 +1,17 @@
+-   repo: git://github.com/pre-commit/pre-commit-hooks
+    sha: v0.8.0
+    hooks:
+    -   id: autopep8-wrapper
+        args:
+        - -i
+        - --ignore=E128,E309,E501
+        exclude: ^docs/.*$
+    -   id: check-json
+    -   id: check-yaml
+    -   id: debug-statements
+    -   id: end-of-file-fixer
+        exclude: ^docs/.*$
+    -   id: trailing-whitespace
+    -   id: pretty-format-json
+        args:
+        - --autofix

UPGRADE-v2.0.md

@@ -278,7 +278,7 @@ Now you can create abstact types super easily, without the need of subclassing t
 class Base(ObjectType):
     class Meta:
         abstract = True
-    
+
     id = ID()
 
     def resolve_id(self, info):

examples/complex_example.py

@@ -22,7 +22,7 @@ def resolve_address(self, info, geo):
 
 
 class CreateAddress(graphene.Mutation):
-    
+
     class Arguments:
         geo = GeoInput(required=True)
 

graphene/pyutils/enum.py

@@ -662,19 +662,25 @@ def __new__(cls, value):
             if member.value == value:
                 return member
     raise ValueError("%s is not a valid %s" % (value, cls.__name__))
+
+
 temp_enum_dict['__new__'] = __new__
 del __new__
 
 
 def __repr__(self):
     return "<%s.%s: %r>" % (
         self.__class__.__name__, self._name_, self._value_)
+
+
 temp_enum_dict['__repr__'] = __repr__
 del __repr__
 
 
 def __str__(self):
     return "%s.%s" % (self.__class__.__name__, self._name_)
+
+
 temp_enum_dict['__str__'] = __str__
 del __str__
 
@@ -705,6 +711,8 @@ def __format__(self, format_spec):
         cls = self._member_type_
         val = self.value
     return cls.__format__(val, format_spec)
+
+
 temp_enum_dict['__format__'] = __format__
 del __format__
 
@@ -751,6 +759,8 @@ def __eq__(self, other):
     if isinstance(other, self.__class__):
         return self is other
     return NotImplemented
+
+
 temp_enum_dict['__eq__'] = __eq__
 del __eq__
 
@@ -759,18 +769,24 @@ def __ne__(self, other):
     if isinstance(other, self.__class__):
         return self is not other
     return NotImplemented
+
+
 temp_enum_dict['__ne__'] = __ne__
 del __ne__
 
 
 def __hash__(self):
     return hash(self._name_)
+
+
 temp_enum_dict['__hash__'] = __hash__
 del __hash__
 
 
 def __reduce_ex__(self, proto):
     return self.__class__, (self._value_, )
+
+
 temp_enum_dict['__reduce_ex__'] = __reduce_ex__
 del __reduce_ex__
 
@@ -785,13 +801,17 @@ def __reduce_ex__(self, proto):
 @_RouteClassAttributeToGetattr
 def name(self):
     return self._name_
+
+
 temp_enum_dict['name'] = name
 del name
 
 
 @_RouteClassAttributeToGetattr
 def value(self):
     return self._value_
+
+
 temp_enum_dict['value'] = value
 del value
 
@@ -817,6 +837,8 @@ def _convert(cls, name, module, filter, source=None):
     module_globals.update(cls.__members__)
     module_globals[name] = cls
     return cls
+
+
 temp_enum_dict['_convert'] = _convert
 del _convert
 

graphene/pyutils/signature.py

@@ -28,7 +28,7 @@ def formatannotation(annotation, base_module=None):
     if isinstance(annotation, type):
         if annotation.__module__ in ('builtins', '__builtin__', base_module):
             return annotation.__name__
-        return annotation.__module__+'.'+annotation.__name__
+        return annotation.__module__ + '.' + annotation.__name__
     return repr(annotation)
 
 
@@ -126,7 +126,7 @@ def signature(obj):
                                                      _partial_kwarg=True)
 
             elif (param.kind not in (_VAR_KEYWORD, _VAR_POSITIONAL) and
-                            not param._partial_kwarg):
+                  not param._partial_kwarg):
                 new_params.pop(arg_name)
 
         return sig.replace(parameters=new_params.values())
@@ -193,11 +193,11 @@ def __repr__(self):
         return '<_ParameterKind: {0!r}>'.format(self._name)
 
 
-_POSITIONAL_ONLY        = _ParameterKind(0, name='POSITIONAL_ONLY')
-_POSITIONAL_OR_KEYWORD  = _ParameterKind(1, name='POSITIONAL_OR_KEYWORD')
-_VAR_POSITIONAL         = _ParameterKind(2, name='VAR_POSITIONAL')
-_KEYWORD_ONLY           = _ParameterKind(3, name='KEYWORD_ONLY')
-_VAR_KEYWORD            = _ParameterKind(4, name='VAR_KEYWORD')
+_POSITIONAL_ONLY = _ParameterKind(0, name='POSITIONAL_ONLY')
+_POSITIONAL_OR_KEYWORD = _ParameterKind(1, name='POSITIONAL_OR_KEYWORD')
+_VAR_POSITIONAL = _ParameterKind(2, name='VAR_POSITIONAL')
+_KEYWORD_ONLY = _ParameterKind(3, name='KEYWORD_ONLY')
+_VAR_KEYWORD = _ParameterKind(4, name='VAR_KEYWORD')
 
 
 class Parameter(object):
@@ -220,11 +220,11 @@ class Parameter(object):
 
     __slots__ = ('_name', '_kind', '_default', '_annotation', '_partial_kwarg')
 
-    POSITIONAL_ONLY         = _POSITIONAL_ONLY
-    POSITIONAL_OR_KEYWORD   = _POSITIONAL_OR_KEYWORD
-    VAR_POSITIONAL          = _VAR_POSITIONAL
-    KEYWORD_ONLY            = _KEYWORD_ONLY
-    VAR_KEYWORD             = _VAR_KEYWORD
+    POSITIONAL_ONLY = _POSITIONAL_ONLY
+    POSITIONAL_OR_KEYWORD = _POSITIONAL_OR_KEYWORD
+    VAR_POSITIONAL = _VAR_POSITIONAL
+    KEYWORD_ONLY = _KEYWORD_ONLY
+    VAR_KEYWORD = _VAR_KEYWORD
 
     empty = _empty
 
@@ -366,7 +366,7 @@ def args(self):
         args = []
         for param_name, param in self._signature.parameters.items():
             if (param.kind in (_VAR_KEYWORD, _KEYWORD_ONLY) or
-                                                    param._partial_kwarg):
+                    param._partial_kwarg):
                 # Keyword arguments mapped by 'functools.partial'
                 # (Parameter._partial_kwarg is True) are mapped
                 # in 'BoundArguments.kwargs', along with VAR_KEYWORD &
@@ -396,7 +396,7 @@ def kwargs(self):
         for param_name, param in self._signature.parameters.items():
             if not kwargs_started:
                 if (param.kind in (_VAR_KEYWORD, _KEYWORD_ONLY) or
-                                                param._partial_kwarg):
+                        param._partial_kwarg):
                     kwargs_started = True
                 else:
                     if param_name not in self.arguments:
@@ -494,7 +494,7 @@ def __init__(self, parameters=None, return_annotation=_empty,
                     params[name] = param
             else:
                 params = OrderedDict(((param.name, param)
-                                                for param in parameters))
+                                      for param in parameters))
 
         self._parameters = params
         self._return_annotation = return_annotation
@@ -604,8 +604,8 @@ def __hash__(self):
 
     def __eq__(self, other):
         if (not issubclass(type(other), Signature) or
-                    self.return_annotation != other.return_annotation or
-                    len(self.parameters) != len(other.parameters)):
+            self.return_annotation != other.return_annotation or
+                len(self.parameters) != len(other.parameters)):
             return False
 
         other_positions = dict((param, idx)
@@ -627,7 +627,7 @@ def __eq__(self, other):
                     return False
                 else:
                     if (idx != other_idx or
-                                    param != other.parameters[param_name]):
+                            param != other.parameters[param_name]):
                         return False
 
         return True
@@ -680,7 +680,7 @@ def _bind(self, args, kwargs, partial=False):
                         parameters_ex = (param,)
                         break
                     elif (param.kind == _VAR_KEYWORD or
-                                                param.default is not _empty):
+                          param.default is not _empty):
                         # That's fine too - we have a default value for this
                         # parameter.  So, lets start parsing `kwargs`, starting
                         # with the current parameter
@@ -730,7 +730,7 @@ def _bind(self, args, kwargs, partial=False):
                 # Signature object (but let's have this check here
                 # to ensure correct behaviour just in case)
                 raise TypeError('{arg!r} parameter is positional only, '
-                                'but was passed as a keyword'. \
+                                'but was passed as a keyword'.
                                 format(arg=param.name))
 
             if param.kind == _VAR_KEYWORD:
@@ -747,8 +747,8 @@ def _bind(self, args, kwargs, partial=False):
                 # parameter, left alone by the processing of positional
                 # arguments.
                 if (not partial and param.kind != _VAR_POSITIONAL and
-                                                    param.default is _empty):
-                    raise TypeError('{arg!r} parameter lacking default value'. \
+                        param.default is _empty):
+                    raise TypeError('{arg!r} parameter lacking default value'.
                                     format(arg=param_name))
 
             else:

graphene/relay/tests/test_node_custom.py

@@ -54,6 +54,7 @@ class Meta:
 class RootQuery(ObjectType):
     node = CustomNode.Field()
 
+
 schema = Schema(query=RootQuery, types=[User, Photo])
 
 

graphene/types/tests/test_base.py

@@ -17,7 +17,7 @@ def __init_subclass_with_meta__(cls, **options):
 def test_basetype():
     class MyBaseType(CustomType):
         pass
-    
+
     assert isinstance(MyBaseType._meta, CustomOptions)
     assert MyBaseType._meta.name == "MyBaseType"
     assert MyBaseType._meta.description is None

graphene/types/tests/test_datetime.py

@@ -55,24 +55,27 @@ def test_time_query():
     assert not result.errors
     assert result.data == {'time': isoformat}
 
+
 def test_bad_datetime_query():
     not_a_date = "Some string that's not a date"
 
     result = schema.execute('''{ datetime(in: "%s") }''' % not_a_date)
-    
+
     assert len(result.errors) == 1
     assert isinstance(result.errors[0], GraphQLError)
     assert result.data == None
 
+
 def test_bad_date_query():
     not_a_date = "Some string that's not a date"
-    
+
     result = schema.execute('''{ date(in: "%s") }''' % not_a_date)
-    
+
     assert len(result.errors) == 1
     assert isinstance(result.errors[0], GraphQLError)
     assert result.data == None
 
+
 def test_bad_time_query():
     not_a_date = "Some string that's not a date"
 
@@ -82,6 +85,7 @@ def test_bad_time_query():
     assert isinstance(result.errors[0], GraphQLError)
     assert result.data == None
 
+
 def test_datetime_query_variable():
     now = datetime.datetime.now().replace(tzinfo=pytz.utc)
     isoformat = now.isoformat()

graphene/types/tests/test_json.py

@@ -10,6 +10,7 @@ class Query(ObjectType):
     def resolve_json(self, info, input):
         return input
 
+
 schema = Schema(query=Query)
 
 

graphene/types/tests/test_uuid.py

@@ -9,6 +9,7 @@ class Query(ObjectType):
     def resolve_uuid(self, info, input):
         return input
 
+
 schema = Schema(query=Query)