-
Notifications
You must be signed in to change notification settings - Fork 8
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
first version of the JSON schema. Fixes issue #27 #133
Conversation
See latest changes @akihikokuroda, @psschwei, and @planetf1 Let's merge, if OK, since I have new version for conditionals |
Other than the last comment. It looks good to me. Thanks! |
OK. Addressed. Resolving all issues. |
OK. We are good for a merge here @akihikokuroda or others. Thx |
Closes issue #132 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Simple JSON workflow that adds the basic workflow features in RC0. One addition is the definition of the "exception" step.
There are two test files in
test/schmas/simple.<yml|json>
Plan is to have tests for issue #132 that will use this file.
You can manually verify the sample tests files using: https://jsonschema.dev
JSON to YAML conversion can be done easily with
json
andyaml
libraries or using: https://www.bairesdev.com/tools/json2yaml/ or similar toolCC: @akihikokuroda @psschwei please do a quick review and merge so I can push the CLI code. Thx