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

add guava explicitly to LS classpath #110

Closed
wants to merge 1 commit into from
Closed

Conversation

lonhutt
Copy link
Contributor

@lonhutt lonhutt commented Apr 9, 2024

fixes #109

Copy link

github-actions bot commented Apr 9, 2024

Test Results

 2 files  ±0   2 suites  ±0   5h 34m 24s ⏱️ - 1d 11h 9m 35s
 7 tests ±0   6 ✅  - 1  0 💤 ±0  1 ❌ +1 
14 runs  ±0  12 ✅  - 2  0 💤 ±0  2 ❌ +2 

For more details on these failures, see this check.

Results for commit 0b76499. ± Comparison against base commit cff08e9.

♻️ This comment has been updated with latest results.

@guw
Copy link
Collaborator

guw commented Apr 9, 2024

We cannot bring in an old version of Guava. This is causing trouble. Instead we have to align the Caffeine library to use the new Guava version.

@guw guw closed this Apr 9, 2024
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.

Need to prepare for Guava update in JDTLS
3 participants