search

prjemian / punx

Python Utilities for NeXus HDF5 files
https://prjemian.github.io/punx
5 stars 7 forks source link

--report option of punx validate is broken? #158

Closed carterbox closed 2 years ago

carterbox commented 2 years ago 
punx validate --report ERROR punx/data/1998spheres.h5

The above command should restrict the report to only showing ERROR status findings? Instead it shows all findings.

Could use:

punx validate punx/data/1998spheres.h5 | grep ERROR

but then why is there a --report option?

prjemian commented 2 years ago

RIght. Can use grep to filter results. The --report option controls the level of finding statuses that are reported. COMMENT,ERROR,NOTE,OK,OPTIONAL,TODO,UNUSED,WARN

Time to revise the handling of this feature.

prjemian commented 2 years ago

On Windows, it is harder to expect the grep command is available and that users know how pipe commands together.