search

ABckh / zed-java-eclipse-jdtls

Eclipse Public License 2.0
29 stars 3 forks source link

Support for Lombok #9

Closed pedrosmmc closed 6 months ago

pedrosmmc commented 6 months ago

Hey collaborators!

First of all, great work so far.

I just want to highly suggest the support for Project Lombok annotations as part of the extension. It is widely adopted.

Regards.

ABckh commented 6 months ago

Hello @pedrosmmc!

Lombok is supported already:

Screenshot 2024-05-30 at 7 05 20 PM Screenshot 2024-05-30 at 7 08 57 PM

Do you have any issues with it?

pedrosmmc commented 6 months ago

Hey ABckh,

Thanks for the prompt reply.

Looks like I have some issues here.

image image image image

Any suggestion?

pedrosmmc commented 6 months ago

Is there any restriction regarding Java version?

image
ABckh commented 6 months ago

This is strange, because I am using exactly the same Java and lombok versions. I would suggest to delete the extension from Zed, then delete java-eclipse-jdtls dir which contains jdtls manually (for mac it is ~/Library/Application Support/Zed/extensions/work/java-eclipse-jdtls) and install it again through Zed.

Let me know if it helps

pedrosmmc commented 6 months ago

Hi ABckh,

It worked, nicely!

The only thing I had to do was remove the JDTLS brew package: https://formulae.brew.sh/formula/jdtls (brew uninstall jdtls)

Thanks. Regards.

ABckh commented 6 months ago

Perfect! By default, jdtls does not support Lombok. In the script for downloading jdtls, I included a step to download the Lombok jar and pass it to jdtls as an extension to provide support.