Skip to content

Commit

Permalink
fix travis: format_map is python 3.2 or later only and ignore new rul…
Browse files Browse the repository at this point in the history
…e W504 (#636)

* format_map is python 3.2 or later only (see also #602)

* Ignore new pycodestyle rule W504

Was introduced in PyCQA/pycodestyle#502 it seems.

Makes travis on master fail https://travis-ci.org/ros-infrastructure/rosdep/jobs/445522297
  • Loading branch information
NikolausDemmel authored and wjwwood committed Nov 1, 2018
1 parent 047b058 commit dc3c601
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/test_flake8.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def test_flake8():
'F841', # ignore presence of unused variables
'I', # ignore import order related warnings
'N802', # ignore presence of upper case in function names
'W504', # ignore line breaks after binary operator (new rule added in 2018)
],
max_line_length=200,
max_complexity=10,
Expand All @@ -53,7 +54,7 @@ def test_flake8():
report._application.formatter.show_statistics(report._stats)
print(
'flake8 reported {report.total_errors} errors'
.format_map(locals()), file=sys.stderr)
.format(**locals()), file=sys.stderr)

assert not report.total_errors, \
'flake8 reported {report.total_errors} errors'.format(**locals())

0 comments on commit dc3c601

Please sign in to comment.