Nils Silbernagel
Nils Silbernagel
A problem I just realized while creating the PR: ignoreError configurations done outside of the baseline would now be put in the baseline as well, which we probably want to...
Also noticed ci failing because a bigger baseline is generated, looking into it
hey @ondrejmirtes thank you for the feedback! That's the approach im currently trying as well. Im basically going through IgnoredErrorHelperResult again with only the ignores from the baseline. One Problem...
Tried a new approach. I did not add the code the baseline formatters but to the generateBaseline function of AnalyseCommand as to not duplicate the logic. To avoid side effects,...
Thanks for the review. Extracted the new logic into BaselineIgnoredErrorHelper, which I added to the config. Deleted the old integration tests to replace them with unit tests.