Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use yaml for buildbot steps #325

Merged
merged 12 commits into from Apr 20, 2016
Prev

Add flake8 lint test for other python files

Currently, the only other python files are the test suite.
  • Loading branch information
aneeshusa committed Apr 20, 2016
commit acf76a1b9625895654145654ea8b5a02ac9c6903
@@ -0,0 +1,19 @@
import os
import subprocess

from tests.util import Failure, Success, project_path


def run():
paths = ['test.py', 'tests']
paths = [os.path.join(project_path(), path) for path in paths]
command = ['flake8'] + paths
ret = subprocess.run(command,
stdout=subprocess.PIPE,
stderr=subprocess.PIPE,
universal_newlines=True)

if ret.returncode == 0:
return Success("Tests passed flake8 lint")
else:
return Failure("Tests failed flake8 lint:", ret.stdout)
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.