rakam-api
rakam-api copied to clipboard
Multiple quality improvements - squid:S00117, squid:S00116
This pull request is focused on resolving occurrences of Sonar rules squid:S00117 - Local variable and method parameter names should comply with a naming convention squid:S00116 - Field names should comply with a naming convention
You can find more information about the issues here: https://dev.eclipse.org/sonar/coding_rules#q=squid:S00117 https://dev.eclipse.org/sonar/coding_rules#q=squid:S00116
Please let me know if you have any questions.
M-Ezzat
I am not sure whether you do not agree with these changes or not. Please close this PR if not.
Oh, yep, this may be happened after fixing the conflicts
You also make some fields static. I think it's hard to guess which variables should be static with automation tools because some of the fields are instance fields because of thread-safety issues. I would rather to keep them as they are.
Ok, reverted