Merge pull request #33 from edmorley/flake8

Add a basic Travis configuration that runs flake8 and 'bob --help'

Author: kennethreitz

.travis.yml

@@ -0,0 +1,22 @@
+sudo: false
+dist: trusty
+language: python
+python:
+  - "2.7"
+  - "3.4"
+  - "3.5"
+  - "3.6"
+install:
+  - pip install .
+  - pip install -r requirements.txt
+script:
+  - flake8
+  # TODO: Replace with an actual test suite:
+  # https://github.com/kennethreitz/bob-builder/issues/31
+  - bob --help
+matrix:
+  allow_failures:
+    - python: "3.4"
+    - python: "3.5"
+    - python: "3.6"
+  fast_finish: true

bob/__init__.py

@@ -1 +1,3 @@
 from . import cli
+
+__all__ = ['cli']

bob/cli.py

@@ -21,9 +21,7 @@
 from .utils import print_stderr
 
 
-
 def build(formula):
-
     f = Formula(path=formula)
 
     try:
@@ -37,10 +35,6 @@ def build(formula):
 
     return f
 
-    # Tarball
-    # Upload to an s3 bucket
-    # Then, sidestep.
-
 
 def deploy(formula, overwrite):
     f = build(formula)
@@ -52,17 +46,14 @@ def deploy(formula, overwrite):
     f.deploy(allow_overwrite=overwrite)
 
 
-
 def main():
-
     args = docopt(__doc__)
 
     formula = args['<formula>']
     do_build = args['build']
     do_deploy = args['deploy']
     do_overwrite = args['--overwrite']
 
-
     if do_build:
         build(formula)
 

bob/models.py

@@ -3,13 +3,15 @@
 from __future__ import print_function
 
 import os
+import re
 import shutil
 import sys
 from tempfile import mkstemp, mkdtemp
 
-import re
+from .utils import (
+    archive_tree, extract_tree, iter_marker_lines, mkdir_p,
+    pipe, print_stderr, process, S3ConnectionHandler)
 
-from .utils import *
 
 WORKSPACE = os.environ.get('WORKSPACE_DIR', 'workspace')
 DEFAULT_BUILD_PATH = os.environ.get('DEFAULT_BUILD_PATH', '/app/.heroku/')
@@ -76,7 +78,6 @@ def depends_on(self):
 
             return depends
 
-
     @property
     def build_path(self):
         """Extracts a declared build path from a given formula."""
@@ -87,7 +88,6 @@ def build_path(self):
         # If none was provided, fallback to default.
         return DEFAULT_BUILD_PATH
 
-
     def resolve_deps(self):
 
         # Dependency metadata, extracted from bash comments.
@@ -155,7 +155,6 @@ def archive(self):
         print('Created: {}'.format(archive))
         self.archived_path = archive
 
-
     def deploy(self, allow_overwrite=False):
         """Deploys the formula's archive to S3."""
         assert self.archived_path

bob/utils.py

@@ -64,6 +64,7 @@ def archive_tree(dir, archive):
         for item in os.listdir(dir):
             tar.add(dir+"/"+item, arcname=item)
 
+
 def extract_tree(archive, dir):
     """Extract tar.gz archive to a given directory."""
     with tarfile.open(archive, 'r:gz') as tar:

requirements.txt

@@ -1,2 +1,3 @@
+flake8==3.3.0
 sphinx
-alabaster
+alabaster

setup.cfg

@@ -0,0 +1,10 @@
+[flake8]
+exclude = .git,__pycache__,docs/
+ignore =
+    # The default ignore list:
+    E121,E123,E126,E226,E24,E704,
+    # Our additions:
+    # E127: continuation line over-indented for visual indent
+    # E128: continuation line under-indented for visual indent
+    # E501: line too long
+    E127,E128,E501