Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: introduces jakarta servlet 6.0 protocol module #448

Merged
merged 3 commits into from
May 2, 2023

Conversation

bvfalcon
Copy link
Contributor

@bvfalcon bvfalcon commented Dec 16, 2022

Changes proposed in this pull request:

As a part of Adaptation to Jakarta EE 10 this PR adds new protocol - Servlet 6.0

Signed-off-by: Vladimir V. Bychkov <github@bychkov.name>
Copy link
Member

@bartoszmajsak bartoszmajsak left a comment

Choose a reason for hiding this comment

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

Thank you for this work. I wonder if we shouldn't rename the package to owning org.jboss.arquillian.protocol.servlet as we have both v_5 and v_6 here. It looks a bit odd right now.

Another thing I was wondering - would it be possible to add a few tests?

@bvfalcon
Copy link
Contributor Author

would it be possible to add a few tests?

Now this new protocol (Servlet 6.0) mainly duplicates already existed "Servlet 5.0", also where is no necessity in additional tests (in addition to Servlet 5.0). May be in future this new protocol becomes more its own functionality and this makes tests useful/necessary.
Nothing to test now: new protocol is only declaration.

@starksm64 starksm64 merged commit 8d3601d into arquillian:master May 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants