add back linters

Author: Julien Nakache

.pre-commit-config.yaml

@@ -7,12 +7,11 @@ repos:
     -   id: check-merge-conflict
     -   id: check-yaml
     -   id: debug-statements
-# TDOO Enable in separate PR
-#     -   id: end-of-file-fixer
-#         exclude: ^docs/.*$
-#     -   id: trailing-whitespace
-#         exclude: README.md
-# -   repo: git://github.com/PyCQA/flake8
-#     rev: 88caf5ac484f5c09aedc02167c59c66ff0af0068  # 3.7.7
-#     hooks:
-#     -   id: flake8
+    -   id: end-of-file-fixer
+        exclude: ^docs/.*$
+    -   id: trailing-whitespace
+        exclude: README.md
+-   repo: git://github.com/PyCQA/flake8
+    rev: 88caf5ac484f5c09aedc02167c59c66ff0af0068  # 3.7.7
+    hooks:
+    -   id: flake8

docs/examples.rst

@@ -37,14 +37,14 @@ Search all Models with Union
     class Query(graphene.ObjectType):
         node = relay.Node.Field()
         search = graphene.List(SearchResult, q=graphene.String())  # List field for search results
-        
+
         # Normal Fields
         all_books = SQLAlchemyConnectionField(BookConnection)
         all_authors = SQLAlchemyConnectionField(AuthorConnection)
 
         def resolve_search(self, info, **args):
             q = args.get("q")  # Search query
-            
+
             # Get queries
             bookdata_query = BookData.get_query(info)
             author_query = Author.get_query(info)
@@ -53,14 +53,14 @@ Search all Models with Union
             books = bookdata_query.filter((BookModel.title.contains(q)) |
                                           (BookModel.isbn.contains(q)) |
                                           (BookModel.authors.any(AuthorModel.name.contains(q)))).all()
-            
+
             # Query Authors
             authors = author_query.filter(AuthorModel.name.contains(q)).all()
 
             return authors + books  # Combine lists
 
     schema = graphene.Schema(query=Query, types=[Book, Author, SearchResult])
-    
+
 Example GraphQL query
 
 .. code::

docs/tips.rst

@@ -31,7 +31,7 @@ If you don't specify any, the following error will be displayed:
 Sorting
 -------
 
-By default the SQLAlchemyConnectionField sorts the result elements over the primary key(s). 
+By default the SQLAlchemyConnectionField sorts the result elements over the primary key(s).
 The query has a `sort` argument which allows to sort over a different column(s)
 
 Given the model

examples/nameko_sqlalchemy/README.md

@@ -51,4 +51,4 @@ Now the following command will setup the database, and start the server:
 
 Now head on over to postman and send POST request to:
 [http://127.0.0.1:5000/graphql](http://127.0.0.1:5000/graphql)
-and run some queries!
+and run some queries!

examples/nameko_sqlalchemy/app.py

@@ -19,7 +19,7 @@ def query(self, request):
               execution_results,
               format_error=default_format_error,is_batch=False, encode=json_encode)
       return result
-  
+
   def parse_body(self,request):
       # We use mimetype here since we don't need the other
       # information provided by content_type
@@ -33,4 +33,4 @@ def parse_body(self,request):
       elif content_type in ('application/x-www-form-urlencoded', 'multipart/form-data'):
           return request.form
 
-      return {}
+      return {}

examples/nameko_sqlalchemy/config.yml

@@ -1 +1 @@
-WEB_SERVER_ADDRESS: '0.0.0.0:5000'
+WEB_SERVER_ADDRESS: '0.0.0.0:5000'

examples/nameko_sqlalchemy/requirements.txt

@@ -1,4 +1,4 @@
 graphene[sqlalchemy]
 SQLAlchemy==1.0.11
 nameko
-graphql-server-core
+graphql-server-core

examples/nameko_sqlalchemy/run.sh

@@ -1,4 +1,4 @@
 #!/bin/sh
 echo "Starting application service server"
 # Run Service
-nameko run --config config.yml service
+nameko run --config config.yml service

examples/nameko_sqlalchemy/service.py

@@ -1,11 +1,11 @@
 #!/usr/bin/env python
 from nameko.web.handlers import http
-from app import App 
+from app import App
 
 
 class DepartmentService:
-    name = 'department'    
+    name = 'department'
 
     @http('POST', '/graphql')
     def query(self, request):
-        return App().query(request)
+        return App().query(request)

graphene_sqlalchemy/tests/test_converter.py

@@ -168,7 +168,7 @@ class Test(Base):
         )
 
     graphene_type = convert_sqlalchemy_column(Test.column)
-    assert graphene_type.kwargs["required"] == False
+    assert not graphene_type.kwargs["required"]
 
 
 def test_should_scalar_list_convert_list():
@@ -343,4 +343,3 @@ def __init__(self, col1, col2):
         )
 
     assert "Don't know how to convert the composite field" in str(excinfo.value)
-