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.

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

Ignore Jakarta Suspended and Context, Support BeanParam #1942

Merged
merged 2 commits into from
Feb 17, 2023

Conversation

JKomoroski
Copy link
Contributor

@JKomoroski JKomoroski commented Feb 17, 2023

A small bit of house keeping to bring the Jakarta package in line with the Jaxrs2 package.

Includes the work from jaxrs2 bean param on pr #1935

Fixes #1940

@velo
Copy link
Member

velo commented Feb 17, 2023

Any ideas on how to keep this 2 modules always in sync?

@velo velo merged commit 0cb3a0a into OpenFeign:master Feb 17, 2023
@JKomoroski
Copy link
Contributor Author

Any ideas on how to keep this 2 modules always in sync?

@velo I don't think there's a great way to handle it as written without getting too fancy with a shade plugin.

Maybe a really generic base class could be extended by jaxrs1, jaxrs2, and jakarta implementations, but this would probably require a new module and the amount of code saved seems minimal to me.

It would almost certainly solve the test dependency quirks we were seeing in jdk8 though

velo added a commit that referenced this pull request Oct 7, 2024
Co-authored-by: Marvin Froeder <velo@users.noreply.github.com>
velo added a commit that referenced this pull request Oct 8, 2024
Co-authored-by: Marvin Froeder <velo@users.noreply.github.com>
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.

JakartaContract does not support JAX-RS 2.x Annotations
2 participants