8000 run pep8 along with other scripts rather than before_script by naved001 · Pull Request #148 · CCI-MOC/m2 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Apr 27, 2022. It is now read-only.

run pep8 along with other scripts rather than before_script #148

Merged
merged 2 commits into from
Dec 2, 2017
Merged

run pep8 along with other scripts rather than before_script #148

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a79d0db
run pep8 along with other scripts rather than before_script
naved001 Sep 27, 2017
04c6b7a
Do a syntax check in the before script as suggested by @jeremyfreudberg
naved001 Oct 3, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,8 @@ services:
- docker

before_script:
# Run pep8 on all .py files in all subfolders
# ignore E402: module level import not at top of file
- find . -name \*.py -exec pep8 --ignore=E402 {} +
# Run syntax checks before spawning the test environment
- find . -name \*.py -exec python -m py_compile {} +

# Run Ceph Container and Wait for some time so that ceph runs
- sudo docker run -d -v ceph:/etc/ceph -e MON_IP=172.17.0.2 -e CEPH_PUBLIC_NETWORK=172.17.0.0/24 --name ceph ceph/demo:tag-build-master-hammer-centos-7
Expand All @@ -34,6 +33,10 @@ before_script:
- sudo docker exec bmi rbd create --image-format 2 --size 10 bmi_test

script:
# Run pep8 on all .py files in all subfolders
# ignore E402: module level import not at top of file
- find . -name \*.py -exec pep8 --ignore=E402 {} +

# Run Tests in BMI container
- sudo docker exec bmi pytest -v --random-order-bucket=global --cov=ims tests/

Expand Down
0