Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Switch to Jersey 3 and jakarta-xml-bind-api#1816

Merged
krisstern merged 3 commits into
jenkinsci:masterfrom
zbynek:update-deps
Oct 12, 2025
Merged

Switch to Jersey 3 and jakarta-xml-bind-api#1816
krisstern merged 3 commits into
jenkinsci:masterfrom
zbynek:update-deps

Conversation

@zbynek

@zbynek zbynek commented Sep 18, 2025

Copy link
Copy Markdown
Contributor

Transition away from javax.* imports where possible to make sure Jenkins doesn't need both javax and jakarta classses at the same time.

Testing done

Automated tests pasing.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@zbynek zbynek requested a review from a team as a code owner September 18, 2025 22:01
@github-actions github-actions Bot added dependencies Pull requests that update a dependency file tests This PR adds/removes/updates test cases labels Sep 18, 2025
@jmini

jmini commented Sep 21, 2025

Copy link
Copy Markdown

FYI as the maintainer of GitLab4J I have been asked in issue gitlab4j/gitlab4j-api#1280 to create a 5.8.1 release that is still on the 5.x.x stream remove the usage of PropertyNamingStrategy to be compatible with newer Jackson version and still be on Jersey 2.

Maybe this version can be useful in this project until you move to 6.x.x

@MarkEWaite

Copy link
Copy Markdown
Contributor

FYI as the maintainer of GitLab4J I have been asked in issue gitlab4j/gitlab4j-api#1280 to create a 5.8.1 release that is still on the 5.x.x stream remove the usage of PropertyNamingStrategy to be compatible with newer Jackson version and still be on Jersey 2.

Maybe this version can be useful in this project until you move to 6.x.x

That would be great! Thanks very much for creating that release! We have multiple issue reports from users that will be happy to have that new release included in the Jenkins plugin.

@jmini

jmini commented Sep 22, 2025

Copy link
Copy Markdown

I think the gitlab4j client library is not used by this plugin. Probably parts of it were copied in this code base (or used as inspiration) a very very long time ago. So my previous comment dot not apply for this repo.

@MarkEWaite :

We have multiple issue reports from users that will be happy to have that new release included in the Jenkins plugin.

Sure. As I wrote somewhere else the report in the upstream project gitlab4j/gitlab4j-api#1280 was really weak and I could not understand the affected version concerned by the issue and the relevance for the jenkins ecosystem (which ironically we are also using in my company). I am glad it now got resolved.

@krisstern krisstern merged commit 077bdc3 into jenkinsci:master Oct 12, 2025
18 checks passed
@zbynek

zbynek commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

@krisstern thanks for merging, I should have mentioned that this should not be released until jersey-3 plugin is released.

@alecharp is trying to get that done, see jenkins-infra/repository-permissions-updater#4663

@zbynek zbynek deleted the update-deps branch October 12, 2025 19:10
@alecharp

Copy link
Copy Markdown
Member

By the way, I released Jersey3-API earlier this week.

@krisstern

Copy link
Copy Markdown
Member

Thanks @zbynek and @alecharp! Let me cut a new release for this plugin later this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file tests This PR adds/removes/updates test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants