18
84%

Ran 25 Jul 2017 02:02PM UTC

Jobs 1

Files 9

Run time 1s

Badge

Embed ▾

pending completion

Build # 18

Build Type

push

travis-ci

Committed by

akosthekiss

Commit Message

Add linting to the testing framework

Pylint's defaults are far from sensible, but at least some of its
checks are useful. The pep8 tool can also be of help. This commit
configures pylint and pep8, fixes some issues discovered by them,
and adds them to the testing framework (tox, Travis CI) to avoid
the fixed issues from creeping back.

Issues fixed:
- logger message formatting (use args to defer substitution),
- eval (avoid),
- import order (standard, custom, local packages),
- indentation,
- type checking (isinstance instead of type),
- regex patterns (raw strings are usually better for escapes).

As a side effect, the commit simplifies tox.ini. The explicit
declaration of the py34, py35, and py36 test environments is
removed as they are provided by tox by default.

Run Details

661 of 728 relevant lines covered (90.8%)

0.91 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
4	18.4 (TOXENV=cov)	25 Jul 2017 02:02PM UTC	0	90.8	Travis Job 18.4

Source Files on build 18

Detailed source file information is not available for this build.

renatahodovan / grammarinator / 18
84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 18

renatahodovan / grammarinator / 18 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 18

renatahodovan / grammarinator / 18
84%

README BADGES
x