Skip to content

Move the one remaining lang2 usage to lang3 - #1237

Merged
nfalco79 merged 1 commit into
jenkinsci:masterfrom
gbhat618:lang2-removal
Jul 23, 2026
Merged

Move the one remaining lang2 usage to lang3#1237
nfalco79 merged 1 commit into
jenkinsci:masterfrom
gbhat618:lang2-removal

Conversation

@gbhat618

@gbhat618 gbhat618 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Remove commons-lang2 usages to prepare for jenkinsci/jenkins#26105.
Lang3 is already a dep in here, so just updating one remaining import path to there.

<dependency>
<groupId>io.jenkins.plugins</groupId>
<artifactId>commons-lang3-api</artifactId>
</dependency>

Testing

Ban enforcer rules failed without the import path update, succeeds afterwards.

@gbhat618 gbhat618 changed the title move to lang3 Move the one remaining lang2 usage to lang3 Jul 23, 2026
@gbhat618
gbhat618 marked this pull request as ready for review July 23, 2026 05:54
@gbhat618

Copy link
Copy Markdown
Contributor Author

/label developer

@nfalco79
nfalco79 merged commit d90ae6a into jenkinsci:master Jul 23, 2026
17 checks passed
@gbhat618
gbhat618 deleted the lang2-removal branch July 23, 2026 10:53
@gbhat618

gbhat618 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

sorry @nfalco79 perhaps we needed to add the developer label to get the release 🤔
it is building on the master branch, lets see if it releases..

Edit: sorry it's not CD enabled. Requesting your help to make a release after master is green, ty.

@timja

timja commented Aug 8, 2026

Copy link
Copy Markdown
Member

Hi @nfalco79 could you please release this its one of the few remaining PRs required for plugin compat testing to pass for the commons-lang2 upgrade, see:

Thanks

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.

4 participants