DoChaoing opened a new pull request, #8022:
URL: https://github.com/apache/incubator-seata/pull/8022

   ### Ⅰ. Describe what this PR did
   
   This PR configures the PMD plugin to output violation messages in English 
instead of Chinese by adding a `.mvn/jvm.config` file with the 
`-Dpmd.language=en` JVM argument.
   
   ### Ⅱ. Does this pull request fix one issue?
   
   Fixes #8021 
   
   ### Ⅲ. Why do we need this change?
   
   The p3c-pmd library (Alibaba Java Coding Guidelines PMD implementation) has 
built-in support for both Chinese and English messages. By default, it uses 
Chinese messages based on the system locale. This PR sets the `pmd.language` 
system property to `en` to ensure English output, making the PMD violations 
more accessible to international contributors.
   
   ### Ⅳ. How to verify it
   
   Run `mvn validate` and observe that PMD violation messages are now in 
English.
   
   Before:
   ```
   [INFO] PMD Failure: 
org.apache.seata.saga.engine.pcext.utils.CompensationHolder:64 
Rule:ConstantFieldShouldBeUpperCaseRule Priority:2 常量【holderLock】命名应全部大写并以下划线分隔.
   ```
   
   After:
   ```
   [INFO] PMD Failure: 
org.apache.seata.saga.engine.pcext.utils.CompensationHolder:64 
Rule:ConstantFieldShouldBeUpperCaseRule Priority:2 Constant [holderLock] should 
be written in upper characters separated by underscores
   ```
   
   ### Ⅴ. Special notes for reviews
   
   This is a configuration-only change that leverages the existing i18n support 
in p3c-pmd. The `pmd.language` system property is documented in the p3c-pmd 
source code (`I18nResources.java`).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to