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.

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

[WFCORE-6237] Eliminating usage of deprecated javax.api aggregation module #5384

Merged
merged 1 commit into from Feb 28, 2023
Merged

[WFCORE-6237] Eliminating usage of deprecated javax.api aggregation module #5384

merged 1 commit into from Feb 28, 2023

Conversation

ropalka
Copy link
Contributor

@ropalka ropalka commented Feb 16, 2023

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Feb 16, 2023
yersan
yersan approved these changes Feb 22, 2023
View reviewed changes
Copy link
Collaborator

@yersan yersan left a comment

Choose a reason for hiding this comment

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

LGTM, I guess we will have another one removing them for WildFly Full and finally, remove the module from the core, correct me if I'm wrong.

@ropalka
Copy link
Contributor Author

ropalka commented Feb 22, 2023

Current plan is to remove usage of deprecated modules.
Then we can decide whether we will ship them (for backward compatibility) or we will drop them.

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Feb 28, 2023
@yersan yersan merged commit 292f400 into wildfly:main Feb 28, 2023
@ropalka ropalka deleted the WFCORE-6237 branch February 28, 2023 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yersan yersan yersan approved these changes

@bstansberry bstansberry Awaiting requested review from bstansberry

Assignees
No one assigned
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Milestone
No milestone
2 participants
@ropalka @yersan