TST Add coverage report to pytest runs

[BenjaminBossan]

Running pytest now gives an output like this:

---------- coverage: platform linux, python 3.10.5-final-0 -----------
Name                              Stmts   Miss  Cover   Missing
---------------------------------------------------------------
skops/__init__.py                     8      1    88%   31
skops/card/__init__.py                2      0   100%
skops/card/_model_card.py            52      0   100%
skops/hub_utils/__init__.py           2      0   100%
skops/hub_utils/_hf_hub.py           98     21    79%   92-94, 125-127, 310, 356-365, 463-473
skops/hub_utils/tests/common.py       1      0   100%
skops/utils/fixes.py                 18      4    78%   13, 22, 56-57
---------------------------------------------------------------
TOTAL                               181     26    86%

[adrinjalali]

Nice!

[adrinjalali]

I'm personally happy w/o them when I run pytest, since certain lines are only run in certain environments, but we can try and remove later if we don't like it.

[BenjaminBossan]

I'm personally happy w/o them when I run pytest, since certain lines are only run in certain environments, but we can try and remove later if we don't like it.

Yes, true. I think it can sometimes be helpful as a developer to see what lines are and aren't covered, e.g. when adding a new module. If you don't need it, just ignore the output, but if you need it, getting the invocation right is a bit tricky so I wanted to configure it for convenience.

[adrinjalali]

I perefer make commands for things which tricky to run, but sometimes needed.

[BenjaminBossan]

make has its place but it can also be kinda rigid sometimes. It really depends on the the situation.

Anyway, since you merged it, I assume it was good enough ;)