search

vheon / JediHTTP

Simple http wrapper around jedi
Apache License 2.0
40 stars 9 forks source link

Update Python in CI environments #28

Closed hexchain closed 7 years ago

hexchain commented 7 years ago
  1. upgrade jedi to latest version to support python 3.6
  2. fix flake8 test
  3. update python versions (and related install scripts) in CI environments

This change is Reviewable

micbou commented 7 years ago

Reviewed 5 of 5 files at r1. Review status: all files reviewed at latest revision, 1 unresolved discussion.

tox.ini, line 16 at r1 (raw file):

deps = {[testenv]deps}
commands =
  flake8 --select=F,C9 --max-complexity=10 --exclude=fixtures jedihttp jedihttp/tests

You can remove the jedihttp/tests argument since it is already covered by the previous one.

Comments from Reviewable

hexchain commented 7 years ago

Fixed.

vheon commented 7 years ago

Thanks for the PR

:lgtm: @homu r+

Reviewed 4 of 5 files at r1, 1 of 1 files at r2. Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

homu commented 7 years ago

:pushpin: Commit bd82220 has been approved by vheon

homu commented 7 years ago

:zap: Test exempted - status