Skip to content

Updated to Lombok based annotation#5

Open
apurbo290 wants to merge 2 commits intozerodha:mainfrom
apurbo290:patch-1
Open

Updated to Lombok based annotation#5
apurbo290 wants to merge 2 commits intozerodha:mainfrom
apurbo290:patch-1

Conversation

@apurbo290
Copy link

Updated to Lombok annotation based Getters and all arg constructor

apurbo290 and others added 2 commits February 22, 2026 11:33
Updated to Lombok annotation based Getters and all arg constructor
@apurbo290 apurbo290 changed the title Update SignatureConfig.java Updated to Lombok based annotation Feb 22, 2026
@joicemjoseph
Copy link
Member

Thoughts:
Lombok adds some friction:

  1. every developer needs to set it up in their IDE
  2. there's a learning curve for anyone new to it
  3. debugging gets trickier since the generated code is invisible
  4. and you're taking on another dependency to maintain.

@iamd3vil
Copy link
Collaborator

The better approach might be to use Java records instead of adding Lombok as a dependency. That will reduce the boilerplate without the need to take on this dependency.

@apurbo290
Copy link
Author

POV:

  1. Regarding another dependency to be maintained - Totally agree, though thoughts differ in Enterprises and maintainers, my thoughts: we should introduce Lombok if repo has future scope and has private artifacts in place.
  2. Regarding Records: If not expecting mutable objs in future as well, my thought also the same.

Thanks!!

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.

3 participants