OpenEnergyPlatform / omi

Repository for the Open Metadata Integration (OMI). For metadata definition see metadata repo:
https://github.com/OpenEnergyPlatform/metadata
GNU Affero General Public License v3.0
7 stars 4 forks source link

Return error report in OMI validation #80

Closed henhuy closed 1 year ago

henhuy commented 1 year ago

Right now, validation stores error report into file. But with this implementation, I cannot inspect error report directly, but have to load report from file again. Instead I suggest to return error report from validation function. In case of no errors an empty error report, stating "no errors found" should be returned. This is the way it is implemented in frictionless as well. As validation function does not return anything ATM, this can be easily implemented. Additionally, I would make storing report on filesystem an option (with default False) as this might not be wanted by users?