Re: [DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
Hey guys, I've just integrates Toolkit and Pack project with SonarCloud, which provides static code analysis. In case you wanna do it with Java Chassis project too, here is the guide: http://servicecomb.apache.org/developers/sonarcloud-how-to/ yhs0092 于2019年7月9日周二 下午4:42写道: > > Yes, it's not so urgent. I'll wait until the weak-type branch get merged. > > > Yours sincerely > > > Yao Haishi > yhs0...@163.com > > > On 7/7/2019 20:37,wjm wjm wrote: > seems not too urgent? > > yhs0092 于2019年7月7日周日 下午12:04写道: > > There are 115 problems pointed out by CheckStyle. The problem distribution > is like below: > - service-registry: 69 > - spring-boot2-starter-servlet: 9 > - spring-boot2-starter-discovery: 1 > - spring-boot-common: 3 > - spring-boot-starter-registry: 1 > - spring-boot-starter-transport: 1 > - spring-cloud-zuul-zipkin: 3 > - spring-boot-starter-discovery: 1 > - foundation-common: 27 > > > The problems are mainly about the name of consts, the constructor of > utility classes, the length of lines, redundant modifiers and magic numbers. > Other problems scanned by findbugs is not counted here. To fix these > problems , there are too many files should be modified. I've created an > issue[1] to track it and wait the weak type work finished. > > > [1]: https://issues.apache.org/jira/browse/SCB-1354 > > > Yours sincerely > > > Yao Haishi > yhs0...@163.com > > > On 7/7/2019 09:03,Willem Jiang wrote: > Hi Haishi, > > Can you list the potential problems which you find? > If they are not the blocker issues, I think we can start to fix them > after the weak type work is finished. > > Willem Jiang > > Twitter: willemjiang > Weibo: 姜宁willem > > On Sat, Jul 6, 2019 at 11:46 AM wjm wjm wrote: > > ohoh, lots of source files? maybe cause very difficult to merge weak type > branch? > > yhs0092 于2019年7月5日周五 下午7:43写道: > > Hi, guys. I've run code static checks for the source code of > ServiceComb-Java-Chassis and find several potential problems. > I want to raise an issue to fix these problems. The changes may be > distributed among a lot of source code files and they are all for code > style and complexity improvement. > Cause I've never handle such kind of issues, is there any thing I should > be aware of? > > > Yours sincerely > > > Yao Haishi > yhs0...@163.com > > > -- Daniel Qian Apache Committer(chanjarster) blog:https://chanjarster.github.io github:https://github.com/chanjarster segmentfault: https://segmentfault.com/u/chanjarster
Re: [DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
Yes, it's not so urgent. I'll wait until the weak-type branch get merged. Yours sincerely Yao Haishi yhs0...@163.com On 7/7/2019 20:37,wjm wjm wrote: seems not too urgent? yhs0092 于2019年7月7日周日 下午12:04写道: There are 115 problems pointed out by CheckStyle. The problem distribution is like below: - service-registry: 69 - spring-boot2-starter-servlet: 9 - spring-boot2-starter-discovery: 1 - spring-boot-common: 3 - spring-boot-starter-registry: 1 - spring-boot-starter-transport: 1 - spring-cloud-zuul-zipkin: 3 - spring-boot-starter-discovery: 1 - foundation-common: 27 The problems are mainly about the name of consts, the constructor of utility classes, the length of lines, redundant modifiers and magic numbers. Other problems scanned by findbugs is not counted here. To fix these problems , there are too many files should be modified. I've created an issue[1] to track it and wait the weak type work finished. [1]: https://issues.apache.org/jira/browse/SCB-1354 Yours sincerely Yao Haishi yhs0...@163.com On 7/7/2019 09:03,Willem Jiang wrote: Hi Haishi, Can you list the potential problems which you find? If they are not the blocker issues, I think we can start to fix them after the weak type work is finished. Willem Jiang Twitter: willemjiang Weibo: 姜宁willem On Sat, Jul 6, 2019 at 11:46 AM wjm wjm wrote: ohoh, lots of source files? maybe cause very difficult to merge weak type branch? yhs0092 于2019年7月5日周五 下午7:43写道: Hi, guys. I've run code static checks for the source code of ServiceComb-Java-Chassis and find several potential problems. I want to raise an issue to fix these problems. The changes may be distributed among a lot of source code files and they are all for code style and complexity improvement. Cause I've never handle such kind of issues, is there any thing I should be aware of? Yours sincerely Yao Haishi yhs0...@163.com
Re: [DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
seems not too urgent? yhs0092 于2019年7月7日周日 下午12:04写道: > There are 115 problems pointed out by CheckStyle. The problem distribution > is like below: > - service-registry: 69 > - spring-boot2-starter-servlet: 9 > - spring-boot2-starter-discovery: 1 > - spring-boot-common: 3 > - spring-boot-starter-registry: 1 > - spring-boot-starter-transport: 1 > - spring-cloud-zuul-zipkin: 3 > - spring-boot-starter-discovery: 1 > - foundation-common: 27 > > > The problems are mainly about the name of consts, the constructor of > utility classes, the length of lines, redundant modifiers and magic numbers. > Other problems scanned by findbugs is not counted here. To fix these > problems , there are too many files should be modified. I've created an > issue[1] to track it and wait the weak type work finished. > > > [1]: https://issues.apache.org/jira/browse/SCB-1354 > > > Yours sincerely > > > Yao Haishi > yhs0...@163.com > > > On 7/7/2019 09:03,Willem Jiang wrote: > Hi Haishi, > > Can you list the potential problems which you find? > If they are not the blocker issues, I think we can start to fix them > after the weak type work is finished. > > Willem Jiang > > Twitter: willemjiang > Weibo: 姜宁willem > > On Sat, Jul 6, 2019 at 11:46 AM wjm wjm wrote: > > ohoh, lots of source files? maybe cause very difficult to merge weak type > branch? > > yhs0092 于2019年7月5日周五 下午7:43写道: > > Hi, guys. I've run code static checks for the source code of > ServiceComb-Java-Chassis and find several potential problems. > I want to raise an issue to fix these problems. The changes may be > distributed among a lot of source code files and they are all for code > style and complexity improvement. > Cause I've never handle such kind of issues, is there any thing I should > be aware of? > > > Yours sincerely > > > Yao Haishi > yhs0...@163.com > > >
Re: [DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
There are 115 problems pointed out by CheckStyle. The problem distribution is like below: - service-registry: 69 - spring-boot2-starter-servlet: 9 - spring-boot2-starter-discovery: 1 - spring-boot-common: 3 - spring-boot-starter-registry: 1 - spring-boot-starter-transport: 1 - spring-cloud-zuul-zipkin: 3 - spring-boot-starter-discovery: 1 - foundation-common: 27 The problems are mainly about the name of consts, the constructor of utility classes, the length of lines, redundant modifiers and magic numbers. Other problems scanned by findbugs is not counted here. To fix these problems , there are too many files should be modified. I've created an issue[1] to track it and wait the weak type work finished. [1]: https://issues.apache.org/jira/browse/SCB-1354 Yours sincerely Yao Haishi yhs0...@163.com On 7/7/2019 09:03,Willem Jiang wrote: Hi Haishi, Can you list the potential problems which you find? If they are not the blocker issues, I think we can start to fix them after the weak type work is finished. Willem Jiang Twitter: willemjiang Weibo: 姜宁willem On Sat, Jul 6, 2019 at 11:46 AM wjm wjm wrote: ohoh, lots of source files? maybe cause very difficult to merge weak type branch? yhs0092 于2019年7月5日周五 下午7:43写道: Hi, guys. I've run code static checks for the source code of ServiceComb-Java-Chassis and find several potential problems. I want to raise an issue to fix these problems. The changes may be distributed among a lot of source code files and they are all for code style and complexity improvement. Cause I've never handle such kind of issues, is there any thing I should be aware of? Yours sincerely Yao Haishi yhs0...@163.com
Re: [DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
Hi Haishi, Can you list the potential problems which you find? If they are not the blocker issues, I think we can start to fix them after the weak type work is finished. Willem Jiang Twitter: willemjiang Weibo: 姜宁willem On Sat, Jul 6, 2019 at 11:46 AM wjm wjm wrote: > > ohoh, lots of source files? maybe cause very difficult to merge weak type > branch? > > yhs0092 于2019年7月5日周五 下午7:43写道: > > > Hi, guys. I've run code static checks for the source code of > > ServiceComb-Java-Chassis and find several potential problems. > > I want to raise an issue to fix these problems. The changes may be > > distributed among a lot of source code files and they are all for code > > style and complexity improvement. > > Cause I've never handle such kind of issues, is there any thing I should > > be aware of? > > > > > > Yours sincerely > > > > > > Yao Haishi > > yhs0...@163.com > > > >
Re: [DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
ohoh, lots of source files? maybe cause very difficult to merge weak type branch? yhs0092 于2019年7月5日周五 下午7:43写道: > Hi, guys. I've run code static checks for the source code of > ServiceComb-Java-Chassis and find several potential problems. > I want to raise an issue to fix these problems. The changes may be > distributed among a lot of source code files and they are all for code > style and complexity improvement. > Cause I've never handle such kind of issues, is there any thing I should > be aware of? > > > Yours sincerely > > > Yao Haishi > yhs0...@163.com > >
[DISCUSSION] About the static check for the source code of ServiceComb-Java-Chassis
Hi, guys. I've run code static checks for the source code of ServiceComb-Java-Chassis and find several potential problems. I want to raise an issue to fix these problems. The changes may be distributed among a lot of source code files and they are all for code style and complexity improvement. Cause I've never handle such kind of issues, is there any thing I should be aware of? Yours sincerely Yao Haishi yhs0...@163.com