feat: introduce flags for line and form coverage thresholds

[krikitti]

Fixes #344

Introduce flags --line-fail-threshold and --form-fail-threshold for the option to handle the two coverages separately. Build will fail if line coverage is below --line-fail-threshold or form coverage is below --form-fail-threshold.

Ignore new flags if --fail-threshold is non-zero in which case the build fails if either line or form coverage is below fail-threshold (same as before).

• You've updated the changelog (if adding/changing user-visible functionality)

[krikitti]

@vemv Cloud you please review this?

[vemv]

It would help us to understand, what would be a typical use case for this?

If there's one that's easy to understand, it would seem worthwhile adding it to the readme

[krikitti]

It would help us to understand, what would be a typical use case for this?

If there's one that's easy to understand, it would seem worthwhile adding it to the readme

@vemv Thanks for getting back to me! The use case for our project is that the form coverage is pretty low at the moment due to some macros, but our line coverage is high enough for our standards. Until we can raise the form coverage (if we ever do that), we would like to set the line and form thresholds separately to check that at least our line coverage is up to standards in all our repos. Apart from the description of the new flags in the readme, should I describe this use case as well?

[vemv]

Thanks!

A brief description like this one should suffice Sometimes, line coverage is high while form coverage isn't as high, so setting a high line coverage requirement can help maintaining an existing standard.

[vemv]

Some suggestions

[krikitti]

@vemv Thanks for the suggestions, please check my latest commits with the added unit test, the updated readme and the added :pre you suggested.

(@imrekoszo fyi)

[vemv]

Looks reasonable to me - great work!

(Note: I don't have write access. I was pinged so here's my review)

[krikitti]

Hi @lvh, do you have write access to this repo? If yes, could you please review this PR? thanks in advance!