Skip to content

[PPP-5790] - remove unused inclusion of commons-lang 2 - #237

Merged
tgf merged 1 commit into
masterfrom
abryant/PPP-5790
Jul 23, 2026
Merged

[PPP-5790] - remove unused inclusion of commons-lang 2#237
tgf merged 1 commit into
masterfrom
abryant/PPP-5790

Conversation

@abryant-hv

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 23, 2026 12:54
@abryant-hv
abryant-hv requested a review from a team as a code owner July 23, 2026 12:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes an unused commons-lang (Apache Commons Lang 2.x) artifact from the driver/client packaging assembly, aligning the packaged dependency set with actual usage (the codebase appears to use commons-lang3 only).

Changes:

  • Removed commons-lang:commons-lang:* from the Maven assembly dependency include list for the driver package.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hitachivantarasonarqube

Copy link
Copy Markdown

@smmribeiro smmribeiro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@buildguy

Copy link
Copy Markdown
Collaborator

👍 Frogbot scanned this pull request and did not find any new security issues.

Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.


@buildguy

Copy link
Copy Markdown
Collaborator

✅ Build finished in 14m 6s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl assemblies/driver

❗ No tests found!

ℹ️ This is an automatic message

@tgf
tgf merged commit b105442 into master Jul 23, 2026
3 checks passed
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.

5 participants