You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2026/08/19 17:33:19 UTC

(maven-reporting-exec) branch master updated: Remove the deprecated localRepository from MavenReportExecutorRequest

This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-reporting-exec.git


The following commit(s) were added to refs/heads/master by this push:
     new eeb9516  Remove the deprecated localRepository from MavenReportExecutorRequest
eeb9516 is described below

commit eeb951683e24d7cc33868214683cfe099d4ceac5
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Sun Aug 16 13:13:03 2026 +0200

    Remove the deprecated localRepository from MavenReportExecutorRequest
    
    Deprecated by MSHARED-1207 and shipped that way since 2.0.0-M10; nothing
    in this project ever read the field, and maven-site-plugin, the only
    consumer in the Maven estate, never set it.
    
    The test keeps org.apache.maven.artifact.repository.ArtifactRepository:
    MavenExecutionRequest still takes one to build the MavenSession, and that
    does not change until the baseline is Maven 4.
---
 .../maven/reporting/exec/MavenReportExecutorRequest.java   | 14 --------------
 .../reporting/exec/TestDefaultMavenReportExecutor.java     |  2 --
 2 files changed, 16 deletions(-)

diff --git a/src/main/java/org/apache/maven/reporting/exec/MavenReportExecutorRequest.java b/src/main/java/org/apache/maven/reporting/exec/MavenReportExecutorRequest.java
index 85016c6..b3638cf 100644
--- a/src/main/java/org/apache/maven/reporting/exec/MavenReportExecutorRequest.java
+++ b/src/main/java/org/apache/maven/reporting/exec/MavenReportExecutorRequest.java
@@ -21,7 +21,6 @@ package org.apache.maven.reporting.exec;
 import java.util.ArrayList;
 import java.util.List;
 
-import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.execution.MavenSession;
 import org.apache.maven.project.MavenProject;
 import org.codehaus.plexus.configuration.xml.XmlPlexusConfiguration;
@@ -35,9 +34,6 @@ import org.codehaus.plexus.util.xml.Xpp3Dom;
  */
 public class MavenReportExecutorRequest {
 
-    @Deprecated
-    private ArtifactRepository localRepository;
-
     private MavenSession mavenSession;
 
     private String executionId;
@@ -46,16 +42,6 @@ public class MavenReportExecutorRequest {
 
     private ReportPlugin[] reportPlugins;
 
-    @Deprecated
-    public ArtifactRepository getLocalRepository() {
-        return localRepository;
-    }
-
-    @Deprecated
-    public void setLocalRepository(ArtifactRepository localRepository) {
-        this.localRepository = localRepository;
-    }
-
     public MavenSession getMavenSession() {
         return mavenSession;
     }
diff --git a/src/test/java/org/apache/maven/reporting/exec/TestDefaultMavenReportExecutor.java b/src/test/java/org/apache/maven/reporting/exec/TestDefaultMavenReportExecutor.java
index 878410f..ae941c0 100644
--- a/src/test/java/org/apache/maven/reporting/exec/TestDefaultMavenReportExecutor.java
+++ b/src/test/java/org/apache/maven/reporting/exec/TestDefaultMavenReportExecutor.java
@@ -158,8 +158,6 @@ public class TestDefaultMavenReportExecutor {
         try {
             MavenReportExecutorRequest mavenReportExecutorRequest = new MavenReportExecutorRequest();
 
-            mavenReportExecutorRequest.setLocalRepository(getLocalRepo());
-
             mavenReportExecutorRequest.setProject(mavenProject);
 
             MavenSession mavenSession = getMavenSession(getLocalRepo(), mavenProject);