Skip to content
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

Fixing squid:S1118 - Utility classes should not have public constructors #62

Conversation

misgersameer
Copy link
Contributor

This pull request is focused on resolving occurrences of Sonar rule "squid:S1118 - Utility classes should not have public constructors". You can find more information about the issue here: https://dev.eclipse.org/sonar/rules/show/squid:S1118

Please let me know if you have any questions.
Sameer Misger

@oldratlee oldratlee force-pushed the master branch 2 times, most recently from c689a57 to dba33a7 Compare February 26, 2016 02:30
oldratlee added a commit that referenced this pull request Mar 7, 2016
…-not-have-public-const

Fixing squid:S1118 -  Utility classes should not have public constructors
@oldratlee oldratlee merged commit eecf7a6 into alibaba:master Mar 7, 2016
oldratlee pushed a commit that referenced this pull request Mar 7, 2016
Utility classes should not have public constructors
oldratlee pushed a commit that referenced this pull request Mar 7, 2016
Utility classes should not have public constructors
@oldratlee oldratlee modified the milestone: 2.0.1 Mar 7, 2016
@oldratlee
Copy link
Member

Merged! Thanks for your contribution!

PS:

Sorry for my dragging action.
I was swinging between code rigorousness and test code readability for a time. 😸

oldratlee pushed a commit that referenced this pull request Jun 11, 2016
Utility classes should not have public constructors
@oldratlee oldratlee self-assigned this Jun 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants