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.

Sign up for GitHub

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

Jump to bottom

[FEATURE] Builder's setter prefix configurable from lombok.config #2596

Open
topr opened this issue Oct 6, 2020 · 3 comments
Open

[FEATURE] Builder's setter prefix configurable from lombok.config #2596

topr opened this issue Oct 6, 2020 · 3 comments

Comments

@topr
Copy link

topr commented Oct 6, 2020 •
edited

As there is the @Builder(setterPrefix="any") option already, it would make sense to have the ability of setting it once across the project at the lombok.config.

Justification:

  • Improved signal-to-noise ratio
  • Better readability (arguably)
  • Helps keeping project convention consistent

BTW. Thanks for adding this option. It really helps preventing name clashes and working with already existing code 👍
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Added possibility to configure @builder setterPref…
e71e69e 
…ix in config file
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
ca9ef77 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
859970c 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
ed8d05e 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
8728047 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
d5ff1f4 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
07b1ed7 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
ae49d67 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
865c634 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue May 20, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
384aec3 
…onfig
@mrwilby
Copy link

mrwilby commented Jul 25, 2021

Would love to see this merged 👍

sadv1r added a commit to sadv1r/lombok that referenced this issue Nov 23, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
ab650e6 
…onfig
sadv1r added a commit to sadv1r/lombok that referenced this issue Nov 23, 2021
@sadv1r
projectlombok#2596 Builder's setter prefix configurable from lombok.c…
a21031a 
…onfig
@HerrDerb
Copy link

HerrDerb commented Sep 23, 2022 •
edited

Would love this feature too! Is there a reason why this isn't a PR yet?

@topr
Copy link
Author

topr commented Dec 12, 2023 •
edited

@HerrDerb actually it was a PR even before your comment had been posted.

Not sure why it is hanging for years instead of being merged 🙄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@topr @mrwilby @HerrDerb