You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Glad to see I'm not the only one staring at SonarQube reports 😮! On a more serious note, I think the first thing to do should be splitting up the large methods into smaller ones. This is guaranteed not to break anything and it could even help to achieve better performance. Reading some of the code currently is quite hard and improving code quality may help more people contribute in the future.
Activity
PhilipRoman commentedon Aug 6, 2018
Glad to see I'm not the only one staring at SonarQube reports 😮! On a more serious note, I think the first thing to do should be splitting up the large methods into smaller ones. This is guaranteed not to break anything and it could even help to achieve better performance. Reading some of the code currently is quite hard and improving code quality may help more people contribute in the future.
Again, thanks for bringing this up!
marci4 commentedon Aug 6, 2018
Yeah we really have to do this!
Helps us and everyone checking all the methods.
Especially the WebSocketServer is a beauty in itself ;)
[-]Improve code quality[/-][+]Improve code quality for 1.4.0[/+]marci4 commentedon Feb 17, 2019
We have a code close for 1.4.0.
Maybe opening another issue for 1.4.1.
Closing this issue as solved!