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.

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

Switch Plexus Annotation to JSR-330 #408

Merged
merged 1 commit into from
Jan 21, 2024
Merged

Switch Plexus Annotation to JSR-330 #408

merged 1 commit into from
Jan 21, 2024

Conversation

slawekjaranowski
Copy link
Member

No description provided.

@olamy
Copy link
Member

olamy commented Jan 16, 2024

comment from the peanut gallery. why changing to something which is deprecated :)
move to jakarta namespace or leave it as is...
but it's really just a comment from the peanut gallery.

@slawekjaranowski
Copy link
Member Author

Step by step. In many place we did such work.
JSR-330 is available in Maven 3.x and classes are not marked as depreciated so we can detect what more else is deprecated.

@olamy
Copy link
Member

olamy commented Jan 16, 2024

You will never have a new release with the classes javax. marked as deprecated as the code is owned by Oracle and they will never touch it.
Eclipse/JarkataEE doesn't have the authority to do it.
The code is just dead!! and so using it is already obsolete (a lot of projects have already moved to jakarta namespace

Step by step. In many place we did such work.

And I probably say the same in many places already :)

But anyway, feel free to do so but it's just double work with changing again to jakarta.* namespace in short/mid term time...

@slachiewicz
Copy link
Member

Maven doesn't support jakarta annotations yet

@slachiewicz slachiewicz added the dependencies Pull requests that update a dependency file label Jan 20, 2024
@slawekjaranowski slawekjaranowski merged commit 3c05d7a into master Jan 21, 2024
26 checks passed
@slawekjaranowski slawekjaranowski deleted the JSR-330 branch January 21, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants