vheon / JediHTTP

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

Add appveyor config #2

Closed vheon closed 9 years ago

vheon commented 9 years ago

Le't go with this. For the moment I'm testing only python 2.7.9 and python 3.3.4 since are preinstalled.

If @Valloric, @puremourning, @oblitum and @micbou want to take a look would be appreciated :+1:

Review on Reviewable

micbou commented 9 years ago

Seems fine to me.

From the documentation, python 3.4 and 3.5 are also pre-installed so you could add them.


Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

Valloric commented 9 years ago

If @micbou is fine with it, I'm with it. :)


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io

puremourning commented 9 years ago

LGTM too


Reviewed 1 of 1 files at r1. Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io