JSONCustomLintr icon indicating copy to clipboard operation
JSONCustomLintr copied to clipboard

Published 20 hours ago • verified publisher icon zsmoore

Lombok Integration

Open ankuagarwal opened this issue 5 years ago • 1 comments

  • Please check if the PR fulfills these requirements
  • [x] Tests for the changes have been added (for bug fixes / features)
  • [x] Docs have been added / updated (for bug fixes / features)

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Feature

  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)? Adding lombok for cleaner code updated documentation

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:

ankuagarwal avatar Jun 04 '19 08:06 ankuagarwal

I don't think we should integrate Lombok. The annotations and changes move a lot towards Kotlin which isn't a bad thing but I would prefer writing this in Kotlin over relying on an outside library to provide language enhancements.

Would prefer to only use Java syntax over bringing in custom annotations and syntax that change how code is written.

zsmoore avatar Jun 05 '19 00:06 zsmoore