This setup, once merged in master, should cause coverage to be evaluated/reported on each PR, along with a green checkmark or a red X depending on whether coverage increased or decreased. In the latter case, the checks technically fail, so while we can still merge those PRs (we shouldn't though), this will nudge us to write tests for any new code that we add.
My guess is that this particular PR won't demonstrate any of this though, but subsequent ones will.
This setup, once merged in
master, should cause coverage to be evaluated/reported on each PR, along with a green checkmark or a red X depending on whether coverage increased or decreased. In the latter case, the checks technically fail, so while we can still merge those PRs (we shouldn't though), this will nudge us to write tests for any new code that we add.My guess is that this particular PR won't demonstrate any of this though, but subsequent ones will.