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 to latest ycmd #2290

Merged
merged 1 commit into from Aug 18, 2016
Merged

[READY] Update to latest ycmd #2290

merged 1 commit into from Aug 18, 2016

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Aug 18, 2016

PR #2279 needs to be merged before this one.


This change is Reviewable

@vheon
Copy link
Contributor

vheon commented Aug 18, 2016

:lgtm:


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


Comments from Reviewable

@Valloric
Copy link
Member

LGTM

@Valloric
Copy link
Member

@homu r=vheon

@homu
Copy link
Contributor

homu commented Aug 18, 2016

📌 Commit 4c040f8 has been approved by vheon

@homu homu merged commit 4c040f8 into ycm-core:master Aug 18, 2016
@homu
Copy link
Contributor

homu commented Aug 18, 2016

⚡ Test exempted - status

homu added a commit that referenced this pull request Aug 18, 2016
[READY] Update to latest ycmd

PR #2279 needs to be merged before this one.

<!-- 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/2290)
<!-- Reviewable:end -->
@micbou micbou deleted the update-ycmd branch August 20, 2016 13:30
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.

None yet

4 participants