plantuml-generator icon indicating copy to clipboard operation
plantuml-generator copied to clipboard

Jakarta namespace update

Open Maxouwell opened this issue 2 years ago • 5 comments

Hello,

Would you take a PR on a jakarta update for addJPAAnnotations and addJavaxAnnotations ?

I'm thinkinf of

  • For addJPAAnnotations, simply adding jakarta.* annotation search in JPAAnalyzerHelper and allowing both on the doc
  • For addJavaxAnnotations, doing one or more of :
    • Renaming the parameter to addJavaBeanAnnotations (breaking change, but I may be the only one using it)
    • Renaming the docs to addJavaBeanAnnotations
    • Adding jakarta.* annotation search in JavaxValidationAnalyzerHelper

What do you think ? Regards

Maxouwell avatar Oct 25 '23 14:10 Maxouwell

An additional consideration of the jakarta namespace makes sense, because sooner or later all users switch to it. That you rename in this case the method addJavaxAnnotations as well as the associated internal methods and the class JavaxValidationAnalyzerHelper likewise. Regarding the breaking change I would like to see a hint in the documentation, so that other potential users know what to change to. In this respect I would adopt the PR.

devlauer avatar Oct 29 '23 11:10 devlauer

Hello !

Not a lot of time on my hand lately, I made a PR for the (easy) JPA Annotation part : https://github.com/devlauer/plantuml-generator/pull/140

Let me know !

Maxouwell avatar Dec 08 '23 14:12 Maxouwell

Sorry for the delayed answer! Thanks you very much for your contribution! I've merged your PR and created a new version (2.4.0), which should be available in the next hours.

devlauer avatar Dec 31 '23 17:12 devlauer

Well it's was not that easy since I botched a part. Fixed in #205

Maxouwell avatar Jun 26 '24 10:06 Maxouwell

And as I was on it, #206 for adding the jakarta Validation part

Maxouwell avatar Jun 27 '24 08:06 Maxouwell

@Maxouwell : Thank you very much for your contribution! And I am really sorry for the long wait! I released both pull requests as part of version 3.0.1. This version should be available within the next hours.

devlauer avatar Aug 18 '24 13:08 devlauer