Skip to content

Commit

Permalink
Run PEP8 check on travis-ci
Browse files Browse the repository at this point in the history
  • Loading branch information
cdeil committed Apr 7, 2013
1 parent f17d98c commit f12e37a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
11 changes: 8 additions & 3 deletions .travis.yml
Expand Up @@ -20,19 +20,21 @@ install:
- time sudo apt-get install -qq python-numpy python-nose python-sphinx python-pexpect python-mongoengine
# Note: we don't install the web packages with apt-get because this resulted
# in version conflicts on travis-ci for WebOB:
# python-pyramid python-magic python-markdown python-weberror python-pastewebkit
# python-pyramid python-magic python-markdown python-weberror python-pastewebkit

- sudo pip install pep8

# Install a ROOT binary that we custom-built in a 64-bit Ubuntu VM
# for the correct Python / ROOT version
- time wget --no-check-certificate https://dl.dropbox.com/u/4923986/rootpy/64/root_v${ROOT}_Python_${TRAVIS_PYTHON_VERSION}.tar.gz
- time tar zxf root_v${ROOT}_Python_${TRAVIS_PYTHON_VERSION}.tar.gz
- source root_v${ROOT}_Python_${TRAVIS_PYTHON_VERSION}/bin/thisroot.sh

script:

# Install WebOOT and dependencies
- python setup.py install --user

script:

# Check if root and PyROOT work ok
- time root -l -q
- time python -c 'import ROOT; ROOT.TBrowser()'
Expand Down Expand Up @@ -60,3 +62,6 @@ script:
# TODO: run weboot unit tests
- cd ..
- python weboot/tests.py

# Check PEP 8 compliance
- make pep8
4 changes: 2 additions & 2 deletions Makefile
@@ -1,5 +1,5 @@
pep8:
pep8 -r weboot setup.py --max-line-length=100
pep8 -r weboot setup.py --max-line-length=100 --count

autopep8:
autopep8 -r weboot setup.py -d --max-line-length=100
autopep8 -r weboot setup.py -i --max-line-length=100

0 comments on commit f12e37a

Please sign in to comment.