Browse Source

Merge branch 'develop' of https://github.com/digitalocean/netbox into develop

Jeremy Stretch 8 years ago
parent
commit
0eb8227044
3 changed files with 12 additions and 1 deletions
  1. 1 0
      .travis.yml
  2. 1 1
      netbox/dcim/tests/test_models.py
  3. 10 0
      scripts/cibuild.sh

+ 1 - 0
.travis.yml

@@ -3,5 +3,6 @@ python:
   - "2.7"
 install:
   - pip install -r requirements.txt
+  - pip install pep8
 script:
   - ./scripts/cibuild.sh

+ 1 - 1
netbox/dcim/tests/test_models.py

@@ -64,7 +64,7 @@ class RackTestCase(TestCase):
             rack=rack1,
             position=10,
             face=RACK_FACE_REAR,
-            )
+        )
         device1.save()
 
         # Validate rack height

+ 10 - 0
scripts/cibuild.sh

@@ -21,6 +21,16 @@ if [[ ! -z $SYNTAX ]]; then
 	EXIT=1
 fi
 
+# Check all python source files for PEP 8 compliance, but explicitly
+# ignore:
+#  - E501: line greater than 80 characters in length
+pep8 --ignore=E501 netbox/
+RC=$?
+if [[ $RC != 0 ]]; then
+	echo -e "\n$(info) one or more PEP 8 errors detected, failing build."
+	EXIT=$RC
+fi
+
 # Prepare configuration file for use in CI
 CONFIG="netbox/netbox/configuration.py"
 cp netbox/netbox/configuration.example.py $CONFIG