Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[READY] Update JediHTTP #888

Merged
merged 1 commit into from
Dec 21, 2017
Merged

[READY] Update JediHTTP #888

merged 1 commit into from
Dec 21, 2017

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Dec 21, 2017

Make JediHTTP shut itself down after 30 minutes of inactivity.

Fixes ycm-core/YouCompleteMe#2852.


This change is Reviewable

Make JediHTTP shut itself down after 30 minutes of inactivity.
@bstaletic
Copy link
Collaborator

:lgtm:


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


Comments from Reviewable

@codecov-io
Copy link

codecov-io commented Dec 21, 2017

Codecov Report

Merging #888 into master will decrease coverage by 0.16%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #888      +/-   ##
==========================================
- Coverage   94.97%   94.81%   -0.17%     
==========================================
  Files          79       41      -38     
  Lines        5457     4014    -1443     
  Branches      172        0     -172     
==========================================
- Hits         5183     3806    -1377     
+ Misses        226      208      -18     
+ Partials       48        0      -48

@puremourning
Copy link
Member

:lgtm: @zzbot r+


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


Comments from Reviewable

@zzbot
Copy link
Contributor

zzbot commented Dec 21, 2017

📌 Commit babf195 has been approved by puremourning

@zzbot
Copy link
Contributor

zzbot commented Dec 21, 2017

⌛ Testing commit babf195 with merge e07d436...

zzbot added a commit that referenced this pull request Dec 21, 2017
[READY] Update JediHTTP

Make JediHTTP shut itself down after 30 minutes of inactivity.

Fixes ycm-core/YouCompleteMe#2852.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/888)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented Dec 21, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: puremourning
Pushing e07d436 to master...

@zzbot zzbot merged commit babf195 into ycm-core:master Dec 21, 2017
zzbot added a commit to ycm-core/YouCompleteMe that referenced this pull request Feb 10, 2018
[READY] Update ycmd

Include the following changes:

 - PR ycm-core/ycmd#789: add support for Windows flags when --driver-mode=cl is given;
 - PR ycm-core/ycmd#848: hide C++ symbols by default;
 - PR ycm-core/ycmd#857: add Java support using jdt.ls;
 - PR ycm-core/ycmd#861: translate libclang error codes to exceptions;
 - PR ycm-core/ycmd#880: support downloading Clang binaries on ARM systems;
 - PR ycm-core/ycmd#883: handle zero column diagnostic from OmniSharp;
 - PR ycm-core/ycmd#884: specify Platform property when compiling OmniSharp;
 - PR ycm-core/ycmd#886: use current working directory in JavaScript completer;
 - PR ycm-core/ycmd#887: update Boost to 1.66.0;
 - PR ycm-core/ycmd#888: update JediHTTP;
 - PR ycm-core/ycmd#889: update Clang to 5.0.1;
 - PR ycm-core/ycmd#891: fix building with system libclang on Gentoo amd64;
 - PR ycm-core/ycmd#904: drop Python 2.6 and Python 3.3 support;
 - PR ycm-core/ycmd#905: calculate the start column when items are not resolved in the language server completer;
 - PR ycm-core/ycmd#912: download Clang binaries from HTTPS;
 - PR ycm-core/ycmd#914: do not try to symlink libclang on Windows.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/youcompleteme/2902)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants