Skip to content

Remove Commons Lang 2 usage - #86

Open
jouni-holsa-pp wants to merge 1 commit into
jenkinsci:masterfrom
jouni-holsa-pp:jouni-holsa-pp-remove-commons-lang-2
Open

Remove Commons Lang 2 usage#86
jouni-holsa-pp wants to merge 1 commit into
jenkinsci:masterfrom
jouni-holsa-pp:jouni-holsa-pp-remove-commons-lang-2

Conversation

@jouni-holsa-pp

Copy link
Copy Markdown

Jenkins 2.579 no longer provides Commons Lang 2 from core, causing the plugin's StringUtils references to fail at runtime. Remove that reliance rather than adding a replacement runtime dependency.

Changes

  • Replace blank checks and capitalization with small Java 8-compatible helpers that preserve Commons Lang 2 behavior.
  • Remove the remaining Commons Lang usage from tests.
  • Add focused regression coverage for the replacement helpers.

Verification

  • mvn test (20 tests passed)
  • Packaged HPI contains no Commons Lang 2 library or class references.

Replace the small StringUtils usage with Java 8-compatible helpers so the plugin works when Jenkins core no longer provides Commons Lang 2.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@jouni-holsa-pp

Copy link
Copy Markdown
Author

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.

1 participant