Skip to content

Add recipe to replace jakarta jws-api with ws-api in EE10#1151

Merged
timtebeek merged 7 commits into
openrewrite:mainfrom
evie-lau:jwsTows
Jun 26, 2026
Merged

Add recipe to replace jakarta jws-api with ws-api in EE10#1151
timtebeek merged 7 commits into
openrewrite:mainfrom
evie-lau:jwsTows

Conversation

@evie-lau

@evie-lau evie-lau commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

In EE10, jakarta.jws-api got consolidated under jakarta.xml.ws-api

Add recipe in EE10 to remove jws-api, and add xml.ws-api

@evie-lau evie-lau self-assigned this Jun 25, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Jun 25, 2026
@timtebeek

Copy link
Copy Markdown
Member

@copilot run ./gradlew rCG

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an OpenRewrite recipe to the Jakarta EE 10 migration set to handle the EE10 consolidation of the retired jakarta.jws:jakarta.jws-api artifact into jakarta.xml.ws:jakarta.xml.ws-api (per #1150).

Changes:

  • Registers a new EE10 migration recipe to remove jakarta.jws-api and ensure jakarta.xml.ws-api is present.
  • Wires the new recipe into the existing UpdateJakartaXmlWsEE10 dependency update chain.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/resources/META-INF/rewrite/jakarta-ee-10.yml Outdated
Comment thread src/main/resources/META-INF/rewrite/jakarta-ee-10.yml
@evie-lau evie-lau changed the title Add recipe to replace jakarta jws-api with ws-api in EE10 #1150 Add recipe to replace jakarta jws-api with ws-api in EE10 Jun 25, 2026

@timtebeek timtebeek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again! Sorry for the at times confusing copilot directions; I'd wanted that to just regenerate the recipes.csv, but mis clicked and then it barfed some feedback. I've polished things up and ready for a merge! We expect a new release midway through next week. Does that timeline suit you there?

@github-project-automation github-project-automation Bot moved this from In Progress to Ready to Review in OpenRewrite Jun 26, 2026
@timtebeek timtebeek merged commit 83f0e68 into openrewrite:main Jun 26, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite Jun 26, 2026
@evie-lau

Copy link
Copy Markdown
Contributor Author

Thanks Tim!
The release timeline should work.

I do have a concern about the change c163f8e from Remove+Add to Change.
This creates seems to create duplicate entries for jakarta.xml.ws-api in the scenario the starting state (pre-EE10) had both jakarta.jws-api and jakarta.xml.ws-api dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

EE10 should replace jakarta.jws-api with jakarta.xml.ws-api

3 participants