Failing CI

[DanielRuf]

It seems CI status is red since a while. =(

https://dev.azure.com/devongovett/devongovett/_build/results?buildId=811

[DeMoorJasper]

Someone was working on getting it fixed, but it appears the PR has been pretty inactive #2532

[mischnic]

but it appears the PR has been pretty inactive

Hey 😄
I have no idea why htmlnano produces no extra linebreaks on windows and whether that's actually the correct behavior (don't think so).

[DeMoorJasper]

@mischnic what about refactoring the tests so it compares the input file size with the minified output file size?
Completely getting rid of the testing linebreaks, as long as it minifies it should be good enough for Parcel afaik.
Testing linebreaks and such goes a bit too far imo as that's what htmlnano should test for already.
In the past it however was the quickest way to verify minification as it used to remove the linebreaks...

I'd be happy to experiment a lil with the tests if you don't have time to try some things atm

[mischnic]

I'd be happy to experiment a lil with the tests if you don't have time to try some things atm

Go ahead! I'd have to setup my Windows machine again.