Nick Mancuso

Closed via https://github.com/checkstyle/checkstyle/pull/12004, addition of this mutator to gui and indentation will only increase execution time with no real benefit.

All approve, label added.

Github, generate report

Link to gist: https://gist.github.com/Vyom-Yadav/4a75df1de3d7f5a1ee9c6e9c66c69542

Due to extensive nature of new pitest docs, I have removed pitest section from "Certain phases..": https://github.com/checkstyle/checkstyle/wiki/How-to-run-certain-phases-and-validations New page is linked from main wiki page, here is direct link: https://github.com/checkstyle/checkstyle/wiki/How-to-Generate-and-Analyze-Pitest-Reports...

In the case of `RequireThisCheck`: Passing `pitest-coding`: ``` org.pitest.mutationtest.engine.gregor.mutators.RemoveConditionalMutator_EQUAL_ELSE >> Generated 904 Killed 894 (99%) > KILLED 881 SURVIVED 10 TIMED_OUT 13 NON_VIABLE 0 > MEMORY_ERROR 0 NOT_STARTED 0 STARTED...

@pbludov I have not seen unstable mutation in CheckstyleAntTask for some time now. Are you good to close this issue?

@pbludov from master build: https://github.com/checkstyle/checkstyle/runs/6893781449?check_suite_focus=true We need to reopen this. @Vyom-Yadav we need to make sure that we keep support of unstable mutations in #11665 .

> @nick-mancuso The mutation above has been addressed, can we close this? please share link to PR where we killed this mutation

Closed via https://github.com/checkstyle/checkstyle/pull/11890