shailpatels opened a new issue, #633:
URL: https://github.com/apache/maven-checkstyle-plugin/issues/633

   ### Affected version
   
   3.6.1-SNAPSHOT
   
   ### Bug description
   
   The custom import order rule 
https://checkstyle.sourceforge.io/checks/imports/customimportorder.html has 
deprecated the `###` as delimiter for the order of packages in the custom 
import order.
   
   A comma separated list can be passed but is not supported by maven 
checkstyle:
   ```
   <module name="Checker">
     <module name="TreeWalker">
       <module name="CustomImportOrder">
         <property name="customImportOrderRules"
           value="STATIC, STANDARD_JAVA_PACKAGE, THIRD_PARTY_PACKAGE"/>
       </module>
     </module>
   </module>
   ``` 
   
   Fails with 
   ```
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-site-plugin:3.20.0:site (default-site) on 
project MCHECKSTYLE-357: Error generating 
maven-checkstyle-plugin:3.6.1-SNAPSHOT:checkstyle report: Failed during 
checkstyle configuration: cannot initialize module TreeWalker - cannot 
initialize module CustomImportOrder - Cannot set property 
'customImportOrderRules' to 'STATIC, STANDARD_JAVA_PACKAGE, 
THIRD_PARTY_PACKAGE': InvocationTargetException: Unexpected rule: STATIC, 
STANDARD_JAVA_PACKAGE, THIRD_PARTY_PACKAGE -> [Help 1]
   ```
   
   For a minimal example you can edit `src/it/MCHECKSTYLE-357/pom.xml` to use 
the new syntax and then run it with
   `mvn invoker:run -Dinvoker.test=MCHECKSTYLE-357` to see the above error.


-- 
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]

Reply via email to