lint python for clean code dcp-682

.flake8

@@ -0,0 +1,47 @@
+[flake8]
+################### FILE PATTERNS ##########################
+
+# Provide a comma-separated list of glob patterns to exclude from checks.
+exclude =
+    .git,
+    __pycache__,
+    .pytest_cache,
+    .mypy_cache,
+    .venv,
+    .vscode,
+    .idea,
+    test
+# Provide a comma-separate list of glob patterns to include for checks.
+filename =
+    *.py
+
+
+################### LINTING ################################
+# Report all errors, even if it is on the same line as a `# NOQA` comment.
+disable-noqa = False
+
+# Set the maximum length that any line (with some exceptions) may be.
+max-line-length = 120
+# Set the maximum allowed McCabe complexity value for a block of code.
+max-complexity = 10
+# Toggle whether pycodestyle should enforce matching the indentation of the opening bracket’s line.
+# incluences E131 and E133
+hang-closing = True
+
+ignore =
+    E128 # continuation line under-indented for visual indent
+    E133 # closing bracket is missing indentation
+    E201 # whitespace after '('
+    E225 # missing whitespace around operator
+    E231 # missing whitespace after ','
+    E241 #multiple spaces after ','
+    E252 #missing whitespace around parameter equals
+    E261 #at least two spaces before inline comment
+    E266 #too many leading '#' for block comment
+    E302 #expected 2 blank lines, found 1
+    E303 #too many blank line
+    E501 # line too long
+    F401 #'io' imported but unused
+    F841 #local variable is assigned to but never used
+    W293 #blank line contains whitespace
+    W391 #blank line at end of file

.pylintrc

@@ -0,0 +1,31 @@
+[GENERAL]
+ignore-paths=
+    .git,
+    __pycache__,
+    .pytest_cache,
+    .mypy_cache,
+    .venv,
+    .vscode,
+    .idea,
+    test
+
+ignore=*.md
+
+[MESSAGES CONTROL]
+disable=all
+enable=
+    R0913
+    C0302, # too many lines
+    R0915, # too many statements
+    R0101, # nested blocks level
+    R0801 # duplicate code
+
+[FORMAT]
+max-line-length=120
+max-module-lines=100
+
+
+[DESIGN]
+max-args=5
+max-statements= 15
+max-nested-blocks=3

.vscode/launch.json

@@ -0,0 +1,15 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "name": "Python: Current File",
+            "type": "python",
+            "request": "launch",
+            "program": "${file}",
+            "console": "integratedTerminal"
+        }
+    ]
+}

.vscode/settings.json

@@ -0,0 +1,11 @@
+{
+    "python.testing.unittestArgs": [
+        "-v",
+        "-s",
+        "./test",
+        "-p",
+        "test_*.py"
+    ],
+    "python.testing.pytestEnabled": false,
+    "python.testing.unittestEnabled": true
+}

README.md

@@ -1,24 +1,31 @@
+# HCA Ingest Broker
+
 [![Build Status](https://travis-ci.org/HumanCellAtlas/ingest-client.svg?branch=master)](https://travis-ci.org/HumanCellAtlas/ingest-broker)
 [![Maintainability](https://api.codeclimate.com/v1/badges/c3cb9256f7e92537fa99/maintainability)](https://codeclimate.com/github/HumanCellAtlas/ingest-broker/maintainability)
 [![Test Coverage](https://api.codeclimate.com/v1/badges/c3cb9256f7e92537fa99/test_coverage)](https://codeclimate.com/github/HumanCellAtlas/ingest-broker/test_coverage)
 [![Docker Repository on Quay](https://quay.io/repository/humancellatlas/ingest-broker/status "Docker Repository on Quay")](https://quay.io/repository/humancellatlas/ingest-broker)
+[![GitHub Super-Linter](https://github.com/ebi-ait/ingest-broker/workflows/Lint%20Code%20Base/badge.svg)](https://github.com/marketplace/actions/super-linter)
 
-# HCA Ingest Broker
 
-Web endpoint for submitting spreadsheets for HCA Ingest and basic admin UI. 
- 
+Web endpoint for submitting spreadsheets for HCA Ingest and basic admin UI.
+
 To run scripts locally you'll need Python 3.6 and all the dependencies in [requirements.txt](requirements.txt).
 
-```
+## Setup
+
+```bash
 pip install -r requirements.txt
 pip install -r requirements-dev.txt
 ```
 
-# Web Application 
+## Configuration
+
+The broker uses a locally running ingest core at `localhost:8000` by default. Connecting to a different one
+Is done by setting the `INGEST_API` environment variable.
 
-## Running with Python 
+## Running with Python
 
-Start the web application with 
+Start the web application with
 
 ```bash
 python broker/broker_app.py
@@ -45,7 +52,7 @@ See the [template](.flaskenv.template).
 See more in [flask's docs](https://flask.palletsprojects.com/en/2.0.x/cli/#environment-variables-from-dotenv)
 
 ## Running With Docker
-Alternatively, you can build and run the app with Docker. To run the web application with Docker for build the Docker image with 
+Alternatively, you can build and run the app with Docker. To run the web application with Docker for build the Docker image with
 
 ```bash
 docker build . -t ingest-broker:latest
@@ -62,9 +69,9 @@ or run against the development Ingest API
 docker run -p 5000:5000 -e INGEST_API=http://api.ingest.dev.data.humancellatlas.org ingest-broker:latest
 ```
 
-The application will be available at http://localhost:5000
+The application will be available at <http://localhost:5000>
 
-# Docs
+## Docs
 
 see [design docs](doc/)
 

requirements-dev.txt

@@ -1,6 +1,11 @@
 -r requirements.txt
 coverage
+flake8
+flake8-eradicate
+flake8-fixme
+flake8-pylint
+pylint
 nose==1.3.7
 pylint
-xlrd
 python-dotenv
+xlrd