-
Notifications
You must be signed in to change notification settings - Fork 5
Update to latest servlet api. #16
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
Conversation
|
Thanks for this! Can you get CI passing? |
|
that should work now, did run mvn verify locally but managed to forget about any action... whelp. |
|
Thanks. I think the action is using some other outdated stuff. Let me know if you want me to get it updated :) |
i just did so in the branch. |
|
well now i'm just confused but i don't use maven often so perhaps mvn verify and mvn test don't quite line up. I'll dig more. |
|
Okay now it should work. |
|
Thanks for doing all this! I'll get a release out shortly. |
|
The release is now on Maven Central: |
|
No problem, thanks for jumping on the PR so fast! |
Changes javax to jakarta namespace.
Changes minimum Java to 11
Also updates Mockito
This should likely just be another branch to run in parallel, however I can't do that with a simple PR.
Fixes #15