-
Notifications
You must be signed in to change notification settings - Fork 833
configure toolchains with toolchain customization #11480
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
Conversation
Signed-off-by: kpenfound <[email protected]>
|
@kpenfound looks good, I would just call the field Also, thinking ahead, will this allow us to also disable checks like we were discussing separately? The only tricky part I see is that |
Signed-off-by: kpenfound <[email protected]>
|
@shykes yeah i was considering |
Signed-off-by: kpenfound <[email protected]>
Signed-off-by: kpenfound <[email protected]>
Signed-off-by: kpenfound <[email protected]>
Signed-off-by: kpenfound <[email protected]>
Signed-off-by: kpenfound <[email protected]>
* configure toolchains with toolchain customization Signed-off-by: kpenfound <[email protected]> * generate docs schema Signed-off-by: kpenfound <[email protected]> * plural customizations Signed-off-by: kpenfound <[email protected]> * keep old config keys for backwards compat Signed-off-by: kpenfound <[email protected]> * generate docs schema Signed-off-by: kpenfound <[email protected]> * preserve arguments and customizations in dagger develop Signed-off-by: kpenfound <[email protected]> * allow deprecated config.Arguments Signed-off-by: kpenfound <[email protected]> --------- Signed-off-by: kpenfound <[email protected]>
* configure toolchains with toolchain customization Signed-off-by: kpenfound <[email protected]> * generate docs schema Signed-off-by: kpenfound <[email protected]> * plural customizations Signed-off-by: kpenfound <[email protected]> * keep old config keys for backwards compat Signed-off-by: kpenfound <[email protected]> * generate docs schema Signed-off-by: kpenfound <[email protected]> * preserve arguments and customizations in dagger develop Signed-off-by: kpenfound <[email protected]> * allow deprecated config.Arguments Signed-off-by: kpenfound <[email protected]> --------- Signed-off-by: kpenfound <[email protected]>
Follow up to #11373
New schema:
{ "name": "app", "engineVersion": "v0.19.4", "toolchains": [ { "name": "hello", "source": "../hello", "customization": [ { "function": ["configurableMessage"], "argument": "message", "default": "hola" } ] } ] }Old schema:
{ "name": "app", "engineVersion": "v0.19.4", "toolchains": [ { "name": "hello", "source": "../hello", "arguments": [ { "function": ["configurableMessage"], "name": "message", "default": "hola" } ] } ] }