Benedek Halasi

Results 5 comments of Benedek Halasi

Verifying that the whitespace is untouched maybe a bit harder than I thought because [`TestMode.TEXT_MATCH` formats both input and output](https://github.com/google/error-prone/blob/31d420b282603d92ad105defed2f47c57ffd1691/test_helpers/src/main/java/com/google/errorprone/BugCheckerRefactoringTestHelper.java#L95). I looked into `doTest` and `TestMode` and it doesn't seem...

Thanks oxkitsune for the `ErrorProneTokens#getTokens(String, Context)`! Apart from the minor questions below, the only big thing to tackle is verifying that this checker reserves whitespaces or deciding that we don't...

> However blocks (both static and instance) are also considered class members: ... Good point, they're indeed non-handled. My reasoning to ignore this case was that using static blocks are...

Hehe, I'll check it out *soon* (this week 🤞) and will update it with the open questions!

Seems PiTest in CI consistently fails in ~1m w/ ``` Failed to execute goal org.pitest:pitest-maven:1.15.3:mutationCoverage (default-cli) on project error-prone-contrib: Execution default-cli of goal org.pitest:pitest-maven:1.15.3:mutationCoverage failed: Coverage generation minion exited abnormally!...