Skip to content

build: Modernize Gradle build infrastructure#2

Open
gsidhwani-nr wants to merge 1 commit into
mainfrom
modernize-gradle-build
Open

build: Modernize Gradle build infrastructure#2
gsidhwani-nr wants to merge 1 commit into
mainfrom
modernize-gradle-build

Conversation

@gsidhwani-nr

Copy link
Copy Markdown
Contributor

Summary

  • Upgraded Gradle wrapper from 7.3 to 8.5 for Java 21 compatibility
  • Replaced root build.gradle with standardized template using verify plugin 4.0 and Maven-based agent resolution
  • Created gradle.properties with java.agent.version=8.6.0
  • Renamed rootProject.name to apache-jackrabbit-instrumentation
  • Updated all modules to use project.findProperty('java.agent.version') and added agent-bridge
  • Replaced sourceCompatibility/targetCompatibility with Java toolchain
  • Changed passes to passesOnly with widened verified ranges and excluded 2.9.0 across all modules to prevent partial instrumentation
  • Replaced compile with implementation in verify blocks
  • Removed newrelic-dependencies.sh
  • Updated release.yml: upgraded JDK to 21, commented out git push

Test plan

  • ./gradlew clean build verifyInstrumentation passes locally
  • CI release workflow runs successfully

- Replace root build.gradle with standardized template (verify plugin 4.0, Maven-based agent resolution)
- Create gradle.properties with java.agent.version=8.6.0
- Upgrade Gradle wrapper from 7.3 to 8.5
- Rename rootProject.name to apache-jackrabbit-instrumentation
- Update all modules to use project.findProperty('java.agent.version') and add agent-bridge
- Replace sourceCompatibility/targetCompatibility with Java toolchain
- Change passes to passesOnly with widened verified ranges:
  - commons: [2.8.5,) exclude 2.9.0
  - spi: [1.4,) exclude 2.9.0
  - data: [2.9.1,)
  - webdav: [2.8.5,) exclude 2.9.0
- Exclude 2.9.0 across all modules to prevent partial instrumentation
- Replace compile with implementation in verify blocks
- Remove newrelic-dependencies.sh
- Update release.yml: upgrade JDK to 21, comment out git push
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