-
Notifications
You must be signed in to change notification settings - Fork 39
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
Have ThrowsCheckedException
also consider supertypes
#914
Conversation
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed. |
68a5d88
to
6a51169
Compare
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed. |
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.
This one only got fully clear again after playing around with the debugger 😄. Nice improvement :).
" }", | ||
"", | ||
" void positive2() {", | ||
" // BUG: Diagnostic contains:", | ||
" callableSink(getClass()::getDeclaredConstructor);", | ||
" callableSink((Callable<?> & Iterable<?>) null);", |
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.
Ohh cool stuff 😄, don't see this often haha.
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.
Nothing stands out to me 👍
This prevents the `MonoFromSupplier` Refaster rule from suggesting non-compilable code.
6a51169
to
ae2ce48
Compare
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed. |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Suggested commit message: