-
Notifications
You must be signed in to change notification settings - Fork 1
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
Move to YAML recipe format #119
Conversation
aaca4f2
to
7e983b4
Compare
@JorgeBornemann I'd appreciate if you could look over the code changes. @Sylviabohnenstengel I'd appreciate it if you could look over the documentation changes. There is a rendered version of the documentation here: https://tmpweb.net/H2iSr5c5Bbb5/ |
This changes the recipe format significantly. No longer is the args table needed for passing through arguments, and the input argument is no longer special. Full documentation will be in a subsequent commit.
It was outputting a thousand lines of progress bar.
This allows dictionaries to be used as argument.
Now any no argument name is special, which means that descriptive names can be used in recipes, increasing readability.
If it wasn't specified then it was failing when a positional argument was passed in, which will always be the case.
b4992d2
to
9783a23
Compare
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.
Great job and the yaml format looks much more intuitive than the toml format.
Should now be ready.
Fixes #107
Fixes #115
Fixes #112