Skip to content
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

add flag to validator to ignore certain warnings #643

Open
pdurbin opened this issue May 2, 2024 · 0 comments
Open

add flag to validator to ignore certain warnings #643

pdurbin opened this issue May 2, 2024 · 0 comments
Assignees

Comments

@pdurbin
Copy link
Member

pdurbin commented May 2, 2024

mlcroissant validate --jsonld croissant.json is extremely helpful but I would like to configure it to ignore certain warnings.

Perhaps we could add an --ignore flag like flake8 has. Here's an example from their docs:

flake8 --ignore=E1,E23,W503 path/to/files/ path/to/more/files/

(By consulting various lists of many possible codes you can see that E1 is about indentation, W503 is about line breaks, etc.)

I'm opening this issue because I see it as a possible solution for this other issue I opened:

That is, I would be happy to ignore the warnings about strings we use for dataset versions.

However, adding a generic --ignore framework is probably a good first step, so I'm opening this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants