1
0
Fork 0

Ported pep8radius addition from mycli.

This commit is contained in:
Irina Truong 2017-04-07 17:00:35 -07:00
parent eab935034e
commit b3ade2db9d
4 changed files with 24 additions and 1 deletions

View File

@ -8,12 +8,15 @@ python:
install:
- pip install . pytest mock codecov==1.5.1 behave pexpect==3.3
- pip install git+https://github.com/hayd/pep8radius.git
script:
- coverage run --source pgcli -m py.test
- cd tests
- behave
- cd ..
# check for pep8 errors, only looking at branch vs master. If there are errors, show diff and return an error code.
- pep8radius master --docformatter --error-status || ( pep8radius master --docformatter --diff; false )
after_success:
- coverage combine

View File

@ -155,3 +155,18 @@ To see stdout/stderr, use the following command:
$ behave --no-capture
PEP8 checks
-----------
When you submit a PR, the changeset is checked for pep8 compliance using
`pep8radius <https://github.com/hayd/pep8radius>`_. If you see a build failing because
of these checks, install pep8radius and apply style fixes:
::
$ pip install pep8radius
$ pep8radius --docformatter --diff # view a diff of proposed fixes
$ pep8radius --docformatter --in-place # apply the fixes
Then commit and push the fixes.

View File

@ -11,6 +11,10 @@ Bug fixes:
* Fixed external editor bug (issue #668). (Thanks: `Irina Truong`_).
* Standardize command line option names. (Thanks: `Russell Davies`_)
Internal changes:
-----------------
* Run pep8 checks in travis (Thanks: `Irina Truong`_).
1.5.1
=====

View File

@ -3,4 +3,5 @@ mock>=1.0.1
tox>=1.9.2
behave>=1.2.4
pexpect==3.3
coverage==4.3.4
coverage==4.3.4
pep8radius