search

vheon / JediHTTP

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

[READY] Add --log option to set logging level #15

Closed micbou closed 8 years ago

micbou commented 8 years ago

Without this option, we can't see the logs with debug level. Also, we need this to pass the ycmd logging level to JediHTTP.

This is mostly copied from ycmd.

This change is Reviewable

vheon commented 8 years ago

:lgtm: @homu r+

Previously, micbou wrote… > #### [READY] Add --log option to set logging level > > Without this option, we can't see the logs with `debug` level. Also, we need this to pass the `ycmd` logging level to `JediHTTP`. > > This is mostly copied from `ycmd`. > >

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

Comments from Reviewable

homu commented 8 years ago

:pushpin: Commit e955b9b has been approved by vheon

homu commented 8 years ago

:zap: Test exempted - status