ReproNim / reproschema-py

Apache License 2.0
2 stars 8 forks source link

[FIX] skip .git and few other files / directory during validation #63

Closed Remi-Gau closed 2 months ago

Remi-Gau commented 2 months ago

Also comes with some refactoring.

Remi-Gau commented 2 months ago

Note that we may get into this weird behavior where validation return "x conforms" even if all files are skipped.

Possible improvement in another PR: run validation on ALL files and only report ALL errors content at the end. As a user I found it annoying to only get errors one after the other when validating large amounts of files.

Remi-Gau commented 2 months ago

FYI: could run it directly on the whole repo of reproschema-library and cobidas_schema.

djarecka commented 2 months ago

thank you, this looks good! Should I merge it or you want to add the exclude parameter?

Remi-Gau commented 2 months ago

let's keep the PR small and add more features in follow up PR, if the current set up has too much friction.