Skip to content

Updated to JDK 21, gradle 8, fixed tests #156

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TareqK
Copy link

@TareqK TareqK commented May 16, 2025

Changes

This is a refactor that upgrades the mvc-common library to Jakarta EE6.0 and JDK 21, allowing use on more modern servers and libraries - the tests were also fixed to account for the expiration of tokens. The main changes were to the build and namespaces of imports - with some changes to the tests using regexes.

References

solves #111

Re-Does #138 because the company that forked it is no longer active.

Testing

  • This change adds test coverage
  • This change has been tested on the latest version of Java or why not

Checklist

Added Context

This will break the current API as it changes all the required args to another package. Will fail CI check for API Changes - but this is expected and part of the issue.

@TareqK TareqK requested a review from a team as a code owner May 16, 2025 18:39
@TareqK TareqK mentioned this pull request May 16, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants