record-builder icon indicating copy to clipboard operation
record-builder copied to clipboard
verified publisher icon Randgalt

Specifying nullability annotation for concrete setters of optional

Open zoehneto opened this issue 9 months ago • 2 comments

Fixes #203 by adding a new optional config option for the desired nullability annotation.

zoehneto avatar May 05 '25 13:05 zoehneto

Nice job, this would be very useful. @Randgalt any news on this?

dosomder avatar May 27 '25 16:05 dosomder

I'll try to get to this soon

Randgalt avatar Jun 02 '25 06:06 Randgalt

@zoehneto I apologize it took me so long on this - I've been slammed at work. I'm just now picking my head up...

I've just pushed a change that I prefer. LMKWYK:

  • Generalize the nullable annotation option so it can be used for other things
  • Have addConcreteSettersForOptional be an enum that has three choices: off, on and on with nullable

Randgalt avatar Aug 10 '25 10:08 Randgalt

@Randgalt thanks for looking at the PR! I really like the changes. Good to merge for me.

zoehneto avatar Aug 11 '25 06:08 zoehneto