Implement retry logic for dealing with compiler errors

[MartinWelgemoed]

Fixes #188

What it does

Now supports mutants that don't compile

How it works

If a compiler error is detected, the mutant on that line is found and marked as non-compiling. A new version of the file is then made without that particular mutant and recompiled.

[hugo-vrijswijk]

Wow, I want to start off by saying how impressive I find it that a first PR to Stryker4s has such significant changes! Not only is this a feature I've wanted for a really long time, but the changes you've made feel like they're made by someone with experience in the codebase. Kudos!!

I have some comments, I hope they're not too incoherent. The main one is about handling of IDs and building up new files without the compile-errored mutants, which, I think, can be done slightly more separated from other pieces of code. A lot of that code is already somewhat dated by now, so don't be afraid to make larger changes.

I am working on Stryker4s for the whole day tomorrow (Friday, 27th of august), so I'd also be happy to take a look and see if I can have a go at some changes if you want. If you want to tackle the comments yourself, be my guest 🙇‍♂️

[MartinWelgemoed]

Thanks a lot for the feedback, I'm also working on Stryker for the whole today so I'll see how far I can get with the suggested changes.

Edit: Also I apparently broke the maven logic somehow, oops I'll fix that too.

[hugo-vrijswijk]

Hey! Very sorry for the delayed response. I haven't been able to make much time to have a proper look again. I'd very much like to have this merged soon. I'll probably have a look at adding some sbt integration tests this Friday.

[hugo-vrijswijk]

Ok! I am really sorry this PR is taking so long to merge. I have 1 small comment, and made some small cleanups myself, but I can't push them to your branch (I think that's something you can enable, but I'm not sure). So I've pushed a commit to a new 946-review branch in stryker-mutator/stryker4s.

With my last comment I think this is ready to merge. Please also ping me if I'm taking too long to respond 🙏

[hugo-vrijswijk]

@MartinWelgemoed 🎉🥳🎈 it's merged. Congratulations! I'm thrilled to finally have this functionality in Stryker4s. When Stryker4s had its first 0.1.0 release, there were two big issues with it: performance and compile errors. The first one was improved earlier this year, and now this! Thanks a lot for starting this PR and having the patience for my reviewing 🙏.