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
FINERACT-822 Update and add more checks to be enabled #975
Conversation
From #971 |
I expect this will fail to build, as per #971. Just do only the adding of the checks here, not don't upgrade, in this PR (or a new one). Adjust title and commit message. |
465ab5f
to
4897a30
Compare
@vorburger Will do only for the checks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (if it passes the build, obviously).
@vorburger am sure this will fail since the checks I added aren't available for ver 2.3.4 |
@vorburger should I update it manually |
"ProtectedMembersInFinalClass", | ||
"StaticAssignmentInConstructor", | ||
"UnescapedEntity", | ||
"EmptyCatch", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems this is not a valid check, maybe a typo or maybe you meant something else and wrote something else. I can merge this once this is resolved.
4897a30
to
5797481
Compare
5797481
to
ae5d7a3
Compare
Since this passes, #971 can be closed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll just merge this now. I don't see why we need to hold this on longer.
FINERACT-822