Skip to content

Fix #695: Apache Commons Email2 #697

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

Merged
merged 1 commit into from
Mar 26, 2025
Merged

Conversation

melloware
Copy link
Contributor

Fix #695: Apache Commons Email2

newVersion: 2.0.x
- org.openrewrite.java.ChangePackage:
oldPackageName: org.apache.commons.mail
newPackageName: org.apache.commons.mail2.jakarta
Copy link

Choose a reason for hiding this comment

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

But org.apache.commons.mail.EmailException is moved to org.apache.commons.mail2.core.EmailException

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK updated to move those 3 specific classes.

Copy link
Contributor

@timtebeek timtebeek left a comment

Choose a reason for hiding this comment

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

Thanks both! Great to have you involved with the project to make this easier for everyone. Also very helpful that you're cross reviewing PRs already.

@github-project-automation github-project-automation bot moved this from In Progress to Ready to Review in OpenRewrite Mar 26, 2025
@timtebeek timtebeek merged commit 35781e1 into openrewrite:main Mar 26, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from Ready to Review to Done in OpenRewrite Mar 26, 2025
@melloware melloware deleted the R695 branch March 26, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

commons-email not migrated in Jakarta EE10 migration
3 participants