Skip to content

build[BACKLOG-50098]: Bump version of aspectjrt dependency to 1.9.25.1#1355

Merged
befc merged 1 commit into
masterfrom
BACKLOG-50098
May 22, 2026
Merged

build[BACKLOG-50098]: Bump version of aspectjrt dependency to 1.9.25.1#1355
befc merged 1 commit into
masterfrom
BACKLOG-50098

Conversation

@befc
Copy link
Copy Markdown
Contributor

@befc befc commented May 21, 2026

Summary

  • remove the repo-local aspectjrt.version override
  • inherit org.aspectj:aspectjrt version 1.9.25.1 from pentaho-parent-pom

Validation

  • mvn -B -nsu -Dmaven.test.skip=true clean install

Copilot AI review requested due to automatic review settings May 21, 2026 17:04
@befc befc requested a review from a team as a code owner May 21, 2026 17:04
@buildguy

This comment has been minimized.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the repository-local override for the AspectJ runtime version so the project can inherit the org.aspectj:aspectjrt version (intended to be 1.9.25.1) from the parent POM lineage.

Changes:

  • Deleted the aspectjrt.version property (1.6.6) from the root pom.xml.

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

Comment thread pom.xml
@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 9m 33s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox

👌 All tests passed!

Tests run: 439, Failures: 0, Skipped: 1    Test Results


ℹ️ This is an automatic message

@hitachivantarasonarqube
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@smmribeiro smmribeiro left a comment

Choose a reason for hiding this comment

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

Looks good!

@befc befc merged commit 2f27037 into master May 22, 2026
6 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.

4 participants