-
Notifications
You must be signed in to change notification settings - Fork 117
Split extra libraries and add more granular control over tests #699
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
… parallel coverage reporting
|
@cehbrecht I'm not certain why the coverage status is taking such a huge hit. I did remove a few useless imports, so it could be that the original values were too high? |
|
@Zeitsperre |
I have split the builds with There were a few tests that I disabled/skipped because there were no assertions made with the results: smoke tests, essentially. It's likely from those changes that the coverage is being impacted. In any case, I think the coverage score is fine. |
Overview
This Pull Request splits out the libraries that support some complex validators (geotiff, netcdf4, GDAL/OGR, zarr) so that basic data types are supported by default.
It also modifies some of the test controls so that
pytestmarkers can be used to explicitly select/deselect tests that rely on these extra libraries.Finally, coverage is now done across all builds in parallel, using
tox-ghand thecoverallsapp/github-actionto remove thecoveralls-pythonlibrary.Related Issue / Discussion
Changes here build on discussions found in #698
Closes #698
Additional Information
There are many more changes I'd like to implement, but I'll limit things here for now until discussion is had in #700.
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)