-
Notifications
You must be signed in to change notification settings - Fork 211
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 JSON schemas #291
base: master
Are you sure you want to change the base?
Add JSON schemas #291
Conversation
@adback03 could I get a review on this? thanks in advance! |
Any updates on this? This looks pretty useful to me. |
Would it help if I added also a GitHub action to check that any change is correct w.r.t. the schemas? |
Can you include a script used to generate the file? Since many of the enum fields are dynamic, they could change frequently. |
@adback03 I used an online generator from one of them, and then run
What I could do is to add a GitHub Action which checks that files pass the schema validation, and reject the PR if they don't. |
This can be useful to check for small typos, and also to auto-generate readers in many programming languages.