xolox / python-executor / 269 / 7
91%
master: 91%

LAST BUILD BRANCH: dev
DEFAULT BRANCH: master
Ran
Files 11
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 91.274%. Remained the same
269.7

push

travis-ci

xolox 
Resolve "new" flake8 warnings*

This concerns the following flake8 warnings which made the merge of pull
request #16 "fail" despite that pull request not touching this code:

  executor/__init__.py:915:17: W504 line break after binary operator
  executor/__init__.py:916:17: W504 line break after binary operator
  executor/__init__.py:1205:17: W504 line break after binary operator
  executor/__init__.py:1206:17: W504 line break after binary operator
  executor/__init__.py:1207:17: W504 line break after binary operator
  executor/__init__.py:1208:17: W504 line break after binary operator
  executor/__init__.py:1375:17: W504 line break after binary operator
  executor/__init__.py:1376:17: W504 line break after binary operator
  executor/tests.py:42:-886: W605 invalid escape sequence '\:'
  executor/contexts.py:951:-77: W605 invalid escape sequence '\s'
  executor/contexts.py:955:24: W605 invalid escape sequence '\s'

Especially W504 is a bit silly and frustrated me at first:

- If you rewrite the code you'll get W503 instead, leading to a rather
  contradictory situation where it's impossible to do the right thing.

- After some hair pulling and reading through [1] it became clear that
  this happens because tox.ini overwrites the built-in ignore list
  instead of appending to it. For more than you probably ever wanted
  to read about that see [1].

[1] https://gitlab.com/pycqa/flake8/issues/466

1297 of 1421 relevant lines covered (91.27%)

0.91 hits per line

Source Files on job 269.7