-
Notifications
You must be signed in to change notification settings - Fork 10
Jakarta Support #9
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
Jakarta Support #9
Conversation
005d3fd
to
500447e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much @nebhale for simplifying everything about Jakarta support!
Since we now deploy to Maven Central, we'll need a bit more XML to add developers, javadoc, licence, url and scm (like I did in the other PR)
But we'll add that with David when we test.
Please let me know about the PR comments I left! I'm curious to know your take on those.
Thanks again!
...te-mapper-jakarta/src/main/java/org/cloudfoundry/router/jakarta/ClientCertificateMapper.java
Outdated
Show resolved
Hide resolved
...te-mapper-jakarta/src/main/java/org/cloudfoundry/router/jakarta/ClientCertificateMapper.java
Outdated
Show resolved
Hide resolved
</plugin> | ||
<plugin> | ||
<groupId>org.jfrog.buildinfo</groupId> | ||
<artifactId>artifactory-maven-plugin</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this plugin utilization is relevant / needed anymore.
With a slight change to deploy.sh and the concourse pipeline , we conform to other Java experience projects and don't rely on plugins not coming from central.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feel free to remove as you desire.
This change updates the project to support both the javax and jakarta namespaces for JEE projects. It does this by creating two separate Maven projects, one for each, which are then shaded into a single JAR used by everyone. The contents of META-INF/services and META-INF/spring.factories route to the correct implementation preventing class loading problems. Signed-off-by: Ben Hale <[email protected]>
Signed-off-by: Ben Hale <[email protected]>
891b3f7
to
209c8cd
Compare
Nudge @anthonydahanne. |
This change updates the project to support both the javax and jakarta namespaces for JEE projects. It does this by creating two separate Maven projects, one for each, which are then shaded into a single JAR used by everyone. The contents of
META-INF/services
andMETA-INF/spring.factories
route to the correct implementation preventing class loading problems.