setuptools icon indicating copy to clipboard operation
setuptools copied to clipboard

[CI] Improve coverage config

Open abravalheri opened this issue 3 years ago • 3 comments

Summary of changes

Currently there is a lot of noise and problems when running coverage in the CI. This is an attempt to improve the configuration and clean the coverage output.

Closes

Pull Request Checklist

abravalheri avatar Jan 25 '22 15:01 abravalheri

Thank you very much @jaraco and @webknjaz for the kind reviews. I tried to address some straight-forward changes as proposed, but others are a bit more complicated.

In the latest commits, I tried to simplify the configuration by removing the [paths] section I had previously added. In general this section is important for testing installed packages, but since setuptools is tested via usedevelop=True, it should not impact too much (the reason why I added it in the first place was because I was doing some experiments to remove usedevelop=True).

Unfortunately I did not find a way of not specifying explicitly the package name in the configuration and still obtain clean output (without explicitly specifying the package names there is a lot of warnings and noise in the output).

abravalheri avatar Feb 08 '22 18:02 abravalheri

Shouldn't this configuration be merged into pyproject.toml?

agronholm avatar Mar 25 '22 10:03 agronholm

Shouldn't this configuration be merged into pyproject.toml?

I haven't yet had a chance to write a blog post about it. Combining all the config into a single file is problemmatic because it conflates concerns and increases the possibility of conflicts as projects share config and have divergent concerns. Let's keep the concerns separated for now. If you think the files should be combined, please file the report at jaraco/skeleton and I can discuss it further there.

jaraco avatar Mar 26 '22 21:03 jaraco