Remove {covr} unneeded from Suggests

[IndrajeetPatil]

No description provided.

[MichaelChirico]

Any reason for removing covr in particular?

To me it's part of Suggests since it should technically be part of our CI suite, though that job appears to be broken for quite some time.

[IndrajeetPatil]

it should technically be part of our CI suite

It is part of our CI suite:

lintr/.github/workflows/test-coverage.yaml

Line 26 in 9f7e4ce

extra-packages: any::covr

It just doesn't need to be a soft dependency, since we are not using any functions from it, IINM.

[MichaelChirico]

I think you're technically right, but I still think it's a good idea to keep it in Suggests. This will help the covr maintainers discover its downstreams / report any issues that might affect us, for example.

[IndrajeetPatil]

discover its downstreams / report any issues that might affect us

But this is what I don't understand. How can it affect us if we use no functionality from it?

We rely on it only via testthat::skip_on_covr(), so it will affect {testthat}, but not us. Am I mistaken about this?

[MichaelChirico]

Filed r-lib/covr#515 to help clarify covr maintainers' preferences. Let's merge this for now.