-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor code according to sonar analysis results #244
Comments
This doesn't really seem like an actionable issue, without details of what could be refactored, or why or how? |
There is plenty of possible refactoring issues that could be done, listing them here has no meaning as they vary a lot. All are provided by sonar (sonarlint plugin) in IDE. My intentions here are that I will do this issue and implement a couple of PRs one by one that will address always a batch of warning/infos from sonar and let those to be discussed whether are worth enough to be merged. Should not break anything and I will learn code base a bit. @robstryker @jeffmaury wdyt? |
Signed-off-by: Andre Dietisheim <[email protected]>
Signed-off-by: Andre Dietisheim <[email protected]>
Signed-off-by: Andre Dietisheim <[email protected]>
Signed-off-by: Andre Dietisheim <[email protected]>
I have run a local sonar analysis from sonarlint eclipse plugin and there are some places that could be refactored according to results. It might help to keep codebase clear.
The text was updated successfully, but these errors were encountered: