opentelemetry-python-contrib
opentelemetry-python-contrib copied to clipboard
Separate `tox.ini` into multiple files
trafficstars
Fixes #2478
This is still a draft PR because I have not yet refactored all packages so that all of them have their own tox.ini file.
The advantages of this proposal are:
- The main
tox.inifile will be much smaller. Right now, this file is extremely big. - The
toxcommands will be much simpler. Instead of runningtox -e py38-test-opentelemetry-name-of-the-packagewe will be able to justcdinto the package directory and then runtox -e test-py38. - This PR will add coverage testing for all packages.
- Right now we run
pytestwith the benchmark options active for all tests, even when only one package here has benchmark tests. This introduces a warning in all of our logs. We should only runpytestwith the benchmark options when there are benchmark tests, this PR fixes this issue. - Right now, we should not run coverage nor benchmark tests in CI (please read the comments below for an explanation). This PR allows us to not run these tests in CI.
- Having a separate
tox.inifile per package allows users to simply runtoxin the package they changed to run all necessary tests.