This is an automated email from the ASF dual-hosted git repository.
hboutemy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-dist-tool.git
commit 6b74f2e651322655923ad3d0f6fc65f6fadf0e79
Author: Hervé Boutemy <hbout...@apache.org>
AuthorDate: Sun Feb 23 18:24:37 2025 +0100
add links to mailing lists
---
.../dist/tools/committers/CommittersStatsReport.java | 7 +++++--
.../org/apache/maven/dist/tools/committers/MLStats.java | 15 ++++++++++++++-
.../apache/maven/dist/tools/committers/MLStatsAnn.java | 6 ++++--
.../maven/dist/tools/committers/MLStatsCommits.java | 8 ++++++--
.../apache/maven/dist/tools/committers/MLStatsIssues.java | 6 ++++--
.../apache/maven/dist/tools/committers/MLStatsVotes.java | 8 ++++++--
6 files changed, 39 insertions(+), 11 deletions(-)
diff --git
a/src/main/java/org/apache/maven/dist/tools/committers/CommittersStatsReport.java
b/src/main/java/org/apache/maven/dist/tools/committers/CommittersStatsReport.java
index 23ce054..5794e63 100644
---
a/src/main/java/org/apache/maven/dist/tools/committers/CommittersStatsReport.java
+++
b/src/main/java/org/apache/maven/dist/tools/committers/CommittersStatsReport.java
@@ -92,7 +92,10 @@ public class CommittersStatsReport extends
AbstractMavenReport {
Map<Committer, List<String>> committerStats =
retrieveCommitterStats();
startSection("Committers Stats");
- paragraph("Committer statistics are based on the searching at
public mailing lists");
+ sink.paragraph();
+ sink.text("Committer statistics are based on the searching at ");
+ link("https://lists.apache.org/list.html?d...@maven.apache.org";,
"public mailing lists");
+ sink.paragraph_();
renderStatsTable(committerStats);
endSection();
@@ -243,7 +246,7 @@ public class CommittersStatsReport extends
AbstractMavenReport {
sink.text(entry.getKey());
sink.definedTerm_();
sink.definition();
- sink.text(entry.getValue().getQueryDescription());
+ entry.getValue().describe(sink);
sink.definition_();
}
diff --git a/src/main/java/org/apache/maven/dist/tools/committers/MLStats.java
b/src/main/java/org/apache/maven/dist/tools/committers/MLStats.java
index 4b1d984..6a7d576 100644
--- a/src/main/java/org/apache/maven/dist/tools/committers/MLStats.java
+++ b/src/main/java/org/apache/maven/dist/tools/committers/MLStats.java
@@ -38,6 +38,7 @@ import com.fasterxml.jackson.core.JsonFactory;
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.core.JsonToken;
import
org.apache.maven.dist.tools.committers.MavenCommittersRepository.Committer;
+import org.apache.maven.doxia.sink.Sink;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -53,10 +54,22 @@ public abstract class MLStats {
entry("d", "lte=1d"), // for stats 1 day is enough
entry("domain", "maven.apache.org"));
- protected abstract String getQueryDescription();
+ protected abstract boolean describeList(Sink sink);
protected abstract List<Map<String, String>> getQueryParamsList(Committer
committer);
+ protected void linkList(Sink sink, String list) {
+ sink.link("https://lists.apache.org/list.html?"; + list +
"@maven.apache.org");
+ sink.text(list);
+ sink.link_();
+ }
+
+ protected void describe(Sink sink) {
+ sink.text("list ");
+ boolean name = describeList(sink);
+ sink.text(" and header_from " + (name ? "committer name" :
"<committerId>@apache.org"));
+ }
+
public String getLast(Committer committer) {
List<Map<String, String>> queryParamsList =
getQueryParamsList(committer);
diff --git
a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsAnn.java
b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsAnn.java
index da60484..4b978ff 100644
--- a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsAnn.java
+++ b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsAnn.java
@@ -25,6 +25,7 @@ import java.util.List;
import java.util.Map;
import
org.apache.maven.dist.tools.committers.MavenCommittersRepository.Committer;
+import org.apache.maven.doxia.sink.Sink;
import static java.util.Map.entry;
import static java.util.Map.ofEntries;
@@ -34,8 +35,9 @@ import static java.util.Map.ofEntries;
public class MLStatsAnn extends MLStats {
@Override
- protected String getQueryDescription() {
- return "list announce and header_from committerId + @apache.org";
+ protected boolean describeList(Sink sink) {
+ linkList(sink, "announce");
+ return false;
}
@Override
diff --git
a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsCommits.java
b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsCommits.java
index 360edf2..65b6997 100644
--- a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsCommits.java
+++ b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsCommits.java
@@ -25,6 +25,7 @@ import java.util.List;
import java.util.Map;
import
org.apache.maven.dist.tools.committers.MavenCommittersRepository.Committer;
+import org.apache.maven.doxia.sink.Sink;
import static java.util.Map.entry;
import static java.util.Map.ofEntries;
@@ -34,8 +35,11 @@ import static java.util.Map.ofEntries;
public class MLStatsCommits extends MLStats {
@Override
- protected String getQueryDescription() {
- return "list commits or site-commits and header_from committerId +
@apache.org";
+ protected boolean describeList(Sink sink) {
+ linkList(sink, "commits");
+ sink.text(" or ");
+ linkList(sink, "site-commits");
+ return false;
}
@Override
diff --git
a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsIssues.java
b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsIssues.java
index 7c5d8f4..2c6b430 100644
--- a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsIssues.java
+++ b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsIssues.java
@@ -25,6 +25,7 @@ import java.util.List;
import java.util.Map;
import
org.apache.maven.dist.tools.committers.MavenCommittersRepository.Committer;
+import org.apache.maven.doxia.sink.Sink;
import static java.util.Map.entry;
import static java.util.Map.ofEntries;
@@ -33,8 +34,9 @@ import static java.util.Map.ofEntries;
@Singleton
public class MLStatsIssues extends MLStats {
- protected String getQueryDescription() {
- return "list issues and header_from committer name";
+ protected boolean describeList(Sink sink) {
+ linkList(sink, "issues");
+ return true;
}
@Override
diff --git
a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsVotes.java
b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsVotes.java
index ed3b6b3..50442d1 100644
--- a/src/main/java/org/apache/maven/dist/tools/committers/MLStatsVotes.java
+++ b/src/main/java/org/apache/maven/dist/tools/committers/MLStatsVotes.java
@@ -25,6 +25,7 @@ import java.util.List;
import java.util.Map;
import
org.apache.maven.dist.tools.committers.MavenCommittersRepository.Committer;
+import org.apache.maven.doxia.sink.Sink;
import static java.util.Map.entry;
import static java.util.Map.ofEntries;
@@ -33,8 +34,11 @@ import static java.util.Map.ofEntries;
@Singleton
public class MLStatsVotes extends MLStats {
- protected String getQueryDescription() {
- return "list dev and header_subject [VOTE] and header_from committer
name";
+ protected boolean describeList(Sink sink) {
+
sink.link("https://lists.apache.org/list?d...@maven.apache.org:lte=1M:&header_subject=[VOTE]";);
+ sink.text("dev and header_subject [VOTE]");
+ sink.link_();
+ return true;
}
@Override
