rnveach

How did we find it now? Did we create this class ourselves?

as long as BDD is in English we will have to force English. We still need to test languages in CI to ensure all messages work.

@romani rebased.

Yes, I only helped with debugging the issue as it seemed interesting, so I added the extra commits to debug/resolve.

> I would rather keep commits that makes it work in our history I agree, I would find some commit that we can reference this PR (or the referenced issue)...

@romani > made one more commit to enforce EN language in execution with ability to override for CI. Is this all that is required for this PR? I am not...

@romani @pbludov Is there any piece of this PR we are going to salvage? If so, can we agree on and identify which to clearly state it.

@timurt Can you help with killing mutations in xpath? Specifically: http://rveach.no-ip.org/checkstyle/regression/pitest-reports/30/pitest-xpath/com.puppycrawl.tools.checkstyle.xpath/ElementNode.java.html http://rveach.no-ip.org/checkstyle/regression/pitest-reports/30/pitest-xpath/com.puppycrawl.tools.checkstyle.xpath/XpathQueryGenerator.java.html#org.pitest.mutationtest.report.html.SourceFile@2125086c_229

@timurt It would be greatly appreciated for those specific areas I mentioned. https://github.com/checkstyle/checkstyle/issues/6320#issuecomment-450595470

New report with only the 1 mutator active: http://rveach.no-ip.org/checkstyle/regression/pitest-reports/39/ Coding package only dropped by 15.