-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
Updated to @codingame/[email protected] #594
Conversation
Yes, you've approved the fix 2 hours ago! I'll trigger a release |
@CGNonofr I should have checked the issue. It states the same problem. 🙈 Thank you! |
@CGNonofr this is ready now. |
…for groovy language server
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@CGNonofr what do you think about the docker-compose approach for the groovy LS? |
It's maybe a little overkill for a single container but hey at least it's easy to use and there's nothing to configure! (I haven't played with it yet though) |
That is the point! 😎 The non-container way is described in the README. |
@CGNonofrinitialize
is broken. I checked your demo differences between1.85.0
and1.85.3
there was no code change required. It should just work with the newer version, right or did I miss something?Version
1.85.4
fixes the issue.The latest version
1.85.5
is now used. In addition this adds a docker-compose based wrapper for the new groovy language server. README is updated.