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

Update jdt.ls to 0.26.0, Java 11 supported. #1122

Merged
merged 1 commit into from
Oct 8, 2018
Merged

Conversation

lsaint
Copy link
Contributor

@lsaint lsaint commented Oct 8, 2018

when jdt.ls version < 0.26.0 :

Java 11 not supported:
Could not determine java version from '11'.
java.lang.IllegalArgumentException: Could not determine java version from '11'.
......


This change is Reviewable

Copy link
Collaborator

@bstaletic bstaletic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR.
Assuming 0.26.0 didn't break any tests, :lgtm:.

Reviewed 1 of 1 files at r1.
Reviewable status: 0 of 2 LGTMs obtained (and 1 stale)

@codecov
Copy link

codecov bot commented Oct 8, 2018

Codecov Report

Merging #1122 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1122   +/-   ##
=======================================
  Coverage   97.58%   97.58%           
=======================================
  Files          89       89           
  Lines        7004     7004           
=======================================
  Hits         6835     6835           
  Misses        169      169

Copy link
Collaborator

@micbou micbou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 1 of 1 files at r1.
Reviewable status: 1 of 2 LGTMs obtained (and 1 stale)

@micbou
Copy link
Collaborator

micbou commented Oct 8, 2018

@zzbot r+

@zzbot
Copy link
Contributor

zzbot commented Oct 8, 2018

📌 Commit cf3c4a4 has been approved by micbou

@zzbot
Copy link
Contributor

zzbot commented Oct 8, 2018

⌛ Testing commit cf3c4a4 with merge 522bf1c...

zzbot added a commit that referenced this pull request Oct 8, 2018
Update jdt.ls to 0.26.0, Java 11 supported.

when jdt.ls version < 0.26.0 :

Java 11 not supported:
Could not determine java version from '11'.
java.lang.IllegalArgumentException: Could not determine java version from '11'.
......

<!-- 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/1122)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented Oct 8, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: micbou
Pushing 522bf1c to master...

@zzbot zzbot merged commit cf3c4a4 into ycm-core:master Oct 8, 2018
@puremourning
Copy link
Member

Thanks for the PR

zzbot added a commit to ycm-core/YouCompleteMe that referenced this pull request Nov 18, 2018
[READY] Update ycmd

Include the following changes:

 - PR ycm-core/ycmd#1080: replace Boost canonical function with our own implementation;
 - PR ycm-core/ycmd#1104: improve filename completer;
 - PR ycm-core/ycmd#1121: support completion FixIts for C-family languages;
 - PR ycm-core/ycmd#1122: update jdt.ls to 0.26.0;
 - PR ycm-core/ycmd#1123: install fixed version of TypeScript in third-party folder;
 - PR ycm-core/ycmd#1124: only add the necessary directories to Python path.

Fixes #3173.

<!-- 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/3174)
<!-- 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