-
Notifications
You must be signed in to change notification settings - Fork 506
Spotless plugin doesn't work with lombok in Eclipse/VS Code/Cursor #2796
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
base: main
Are you sure you want to change the base?
Conversation
| return new ByteBuddy() | ||
| .subclass(Object.class) | ||
| .name(name) | ||
| .defineMethod("augment", name.equals("lombok.core.FieldAugment") ? Object.class : Object.class, Visibility.PUBLIC, Ownership.STATIC) |
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.
What is the point of name.equals("lombok.core.FieldAugment") ? Object.class : Object.class given that both branches return the same value?
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.
I have fixed it.
|
i'm trying to fix this up to merge, please don't force-push |
| } | ||
|
|
||
| dependencies { | ||
| implementation 'net.bytebuddy:byte-buddy:1.18.3' |
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.
we have a lib and lib-extra division so that lib has no dependencies, and lib-extra gets stuck with the dependency hell.
byte-buddy cannot be a dependency here. I don't fully understand what you're doing here, but you'll have to do it without adding dependencies into lib.
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.
@nedtwigg I will try.
Fixes #2795