Skip to content

Improve code quality for 1.4.0 #749

Closed

Description

@marci4

Expected Behavior

SonarQube should not report to a lot of errors.

Current Behavior

Currently SonarQube reports some errors. These should be fixed!

Possible Solution

Fix errors found or disable rules!

Context

Code quality

Activity

  1. added this to the milestone on Aug 6, 2018
  2. PhilipRoman commented on Aug 6, 2018

    @PhilipRoman
    Collaborator

    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!

  3. marci4 commented on Aug 6, 2018

    @marci4
    CollaboratorAuthor

    Yeah we really have to do this!
    Helps us and everyone checking all the methods.
    Especially the WebSocketServer is a beauty in itself ;)

  4. modified the milestones: , Release 1.4.0 on Sep 6, 2018
  5. changed the title [-]Improve code quality[/-] [+]Improve code quality for 1.4.0[/+] on Feb 17, 2019
  6. marci4 commented on Feb 17, 2019

    @marci4
    CollaboratorAuthor

    We have a code close for 1.4.0.

    Maybe opening another issue for 1.4.1.

    Closing this issue as solved!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions