Re: [ofbiz-framework] branch trunk updated: Improved: Handle remaining checkstyle errors (OFBIZ-12169)

2021-05-25 Thread Jacques Le Roux
Hi, Thanks to nmancus1's help at https://github.com/checkstyle/checkstyle/issues/10012, I finally don't know if it's a checkstyle config issue or a bug in checkstyle. I doubt it's the later. As I wrote there I'm unsure about what to do. Maybe the best is to keep the current checkstyle config

Re: [ofbiz-framework] branch trunk updated: Improved: Handle remaining checkstyle errors (OFBIZ-12169)

2021-05-17 Thread Jacques Le Roux
Hi, I send a bug report: Whitespaces at the head of line before a cast and after an instanceof or an "enhanced for" results to "'(' should be on the previous line"error · Issue #10012 · checkstyle/checkstyle (github.com) BTW, I have

Re: [ofbiz-framework] branch trunk updated: Improved: Handle remaining checkstyle errors (OFBIZ-12169)

2021-05-04 Thread jler...@apache.org
Hi Team, I have been working on https://checkstyle.sourceforge.io/config_whitespace.html#MethodParamPad I could fix few issues in code. But there are still some cases that can't be handled by changing code, notably when a casting is necessary, and not only. I believe there is an error in