Skip to content

org.glassfish.jersey.xxx:xxx versions not updated when using ...migrate.jakarta.JakartaEE10 #682

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
Closed
jepsar opened this issue Mar 19, 2025 · 2 comments · Fixed by #684
Closed

org.glassfish.jersey.xxx:xxx versions not updated when using ...migrate.jakarta.JakartaEE10 #682

jepsar opened this issue Mar 19, 2025 · 2 comments · Fixed by #684
Assignees
@melloware
Labels
bug Something isn't working

Comments

@jepsar
Copy link

jepsar commented Mar 19, 2025

I ran

mvn -U org.openrewrite.maven:rewrite-maven-plugin:run \
    -Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-migrate-java:RELEASE \
    -Drewrite.activeRecipes=org.openrewrite.java.migrate.jakarta.JakartaEE10

and found that the dependencies

      <groupId>org.glassfish.jersey.xxx</groupId>
      <artifactId>xxx</artifactId>
      <version>2.x</version>

were not updated to the EE10 compatible version

      <groupId>org.glassfish.jersey.xxx</groupId>
      <artifactId>xxx</artifactId>
      <version>3.1.10</version>

CC @melloware

GroupIds I've found (not sure how generic the rules can be):

  • org.glassfish.jersey.core
  • org.glassfish.jersey.inject
  • org.glassfish.jersey.media
  • org.glassfish.jersey.containers
  • org.glassfish.jersey.server
  • org.glassfish.jersey.ext
@jepsar jepsar added the bug Something isn't working label Mar 19, 2025
@melloware
Copy link
Contributor

@timtebeek also assign to me please

@timtebeek timtebeek moved this to Backlog in OpenRewrite Mar 19, 2025
melloware added a commit to melloware/rewrite-migrate-java that referenced this issue Mar 19, 2025
@melloware
Fix openrewrite#682: EE10 migration Jersey to 3.1.x
b569dc8
@melloware melloware mentioned this issue Mar 19, 2025
Merged
@melloware
Copy link
Contributor

PR submitted

timtebeek added a commit that referenced this issue Mar 19, 2025
@melloware @timtebeek
Fix #682: EE10 migration Jersey to 3.1.x (#684)
06893d0 
* Fix #682: EE10 migration Jersey to 3.1.x

* Fix punctuation in Jakarta EE 10 YAML

---------

Co-authored-by: Tim te Beek <[email protected]>
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenRewrite Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@melloware melloware

Labels
bug Something isn't working
Projects
OpenRewrite
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #682: EE10 migration Jersey to 3.1.x melloware/rewrite-migrate-java
2 participants
@melloware @jepsar