Nico Jansen

Results 436 comments of Nico Jansen

> small signals library right into the `metrics` package Yeah, it turned out to be wishful thinking on my part. It cannot be 'very small', as we also need a...

> If I understand correctly, the main purpose would be the expensive recalculation of the metrics on large reports, even though not much has actually changed. I don't really see...

Yeah, I definitely see your point. 3 solutions here: 1. Add something like `"reportAppearance"` to the schema. I don't like this solution personally. 2. We could add it as an...

Yes, i think this is pretty straightforward to add. I would suggest adding a line number and focussed mutant/test to the url fragment.

I think we should call it `testRunnerOutput`. @hugo-vrijswijk @rouke-broersma, thoughts?

Yeah, good questions. I thought it was for mutants only. Probably @maks-rafalko will be able to answer in detail.

That's very useful yes! I think the easiest solution might be to put it in the horizontal drawer, as it is designed for a lot of text.

You're not the first one with this feedback, and I kind of agree. Feel free to make a mockup of the diff view in combination with surviving and killed mutants.

> The new indicators do not show the mutation ids. It might be nice if the ids showed up in their hovers. I disagree. Mutant id's can be any string....

Thanks for the feedback @commonquail . I'm responsible for the new underlining, so sorry about that. I did have the support of others as well 😅 > absolutely terrible for...