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

michaelo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new 142304a  [MCHECKSTYLE-450] Checkstyle rule link format results in 404
142304a is described below

commit 142304a4d93d41a2f95e5f577387ca9de004807a
Author: rshindo <ryo.shi...@gmail.com>
AuthorDate: Fri May 3 15:35:57 2024 +0900

    [MCHECKSTYLE-450] Checkstyle rule link format results in 404
    
    New links direct to https://checkstyle.org/checks/{category}/{ruleName}.html
    
    This fixes #134
---
 .../org/apache/maven/plugins/checkstyle/CheckstyleReportRenderer.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleReportRenderer.java
 
b/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleReportRenderer.java
index b1dd166..3db3497 100644
--- 
a/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleReportRenderer.java
+++ 
b/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleReportRenderer.java
@@ -242,7 +242,8 @@ public class CheckstyleReportRenderer extends 
AbstractMavenReportRenderer {
         // column 2: Rule name + configured attributes
         sink.tableCell();
         if (!"extension".equals(category)) {
-            sink.link("https://checkstyle.org/config_"; + category + ".html#" + 
ruleName);
+            sink.link(
+                    "https://checkstyle.org/checks/"; + category + "/" + 
ruleName.toLowerCase(Locale.ENGLISH) + ".html");
             sink.text(ruleName);
             sink.link_();
         } else {

Reply via email to