Roman Ivanov
Roman Ivanov
Do checkout of io repo by pat, not a checkstyle.
We need to preserve changes at https://github.com/checkstyle/checkstyle/pull/11529/commits they show correct way to proporate locale, please squash them in one commit Second commit should be from Kirill to enforce EN locale...
@Vyom-Yadav , @nick-mancuso is this known instability, can we register it in suppression xml ? to not bother us unexpectedly
based my thoughts at https://github.com/checkstyle/checkstyle/pull/11529#issuecomment-1122616467 I placed one more commit with Albanian language - https://www.oracle.com/java/technologies/javase/jdk8-jre8-suported-locales.html we unlikely to get translation to this language :), so it will be prove that...
Does it make sense to collect all instabilities? to notice that they likely to happen, and may be not that often. Do you have already some issue for this ?
``` Pitest-api New surviving mutation(s) found: Source File: "Violation.java" Class: "com.puppycrawl.tools.checkstyle.api.Violation$Utf8Control" Method: "newBundle" Line Contents: "final String resourceName = toResourceName(bundleName, "properties");" Mutator: "org.pitest.mutationtest.engine.gregor.mutators.experimental.ArgumentPropagationMutator" Description: "replaced call to com/puppycrawl/tools/checkstyle/api/Violation$Utf8Control::toResourceName with argument"...
Strange , stable failure. ``` Source File: "Violation.java" Class: "com.puppycrawl.tools.checkstyle.api.Violation$Utf8Control" Method: "newBundle" Line Contents: "final String resourceName = toResourceName(bundleName, "properties");" Mutator: "org.pitest.mutationtest.engine.gregor.mutators.experimental.ArgumentPropagationMutator" Description: "replaced call to com/puppycrawl/tools/checkstyle/api/Violation$Utf8Control::toResourceName with argument" Line...
@Vyom-Yadav , Should we add this survival to suppression file ?
is it possible to just violate situation that there is static import for same method from different classes? so one of them is definitely extra, and user will decide.
@rnveach , thanks a lot for investigation. I though it is not possible :). @stoyanK7, please confirm rnveach example, if this is true, we can not support it. So we...