search

ioos / compliance-checker

Python tool to check your datasets against compliance standards
http://ioos.github.io/compliance-checker/
Apache License 2.0
108 stars 58 forks source link

Final merge prior to 5.1.1 release #1062

Closed benjwadams closed 5 months ago

benjwadams commented 5 months ago

Something possibly looks awry with compliance checker plugins. Investigating.

ocefpaf commented 5 months ago

Something possibly looks awry with compliance checker plugins. Investigating.

Can you elaborate? The plugins tests are passing. The failure I can see is in 

compliance_checker/tests/test_cli.py::TestCLI::test_multi_checker_return_value FAILED
benjwadams commented 5 months ago

pre-commit.ci fix

benjwadams commented 5 months ago

@ocefpaf, uncovered a last minute bug in valid_range checks. I've forgotten how to issue the pre-commit black formatting fixes via GitHub Issues comments. Once this code has your approval as a reviewer, I will merge and draft a release, kicking off the PyPI build.

ocefpaf commented 5 months ago

pre-commit.ci autofix

ocefpaf commented 5 months ago

@ocefpaf, uncovered a last minute bug in valid_range checks. I've forgotten how to issue the pre-commit black formatting fixes via GitHub Issues comments. Once this code has your approval as a reviewer, I will merge and draft a release, kicking off the PyPI build.

I have a minor question above on the cfutil change but it is good to go IMO.

benjwadams commented 5 months ago

pre-commit.ci autofix