Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@kpenfound
Copy link
Contributor

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"
        }
      ]
    }
  ]
}

@kpenfound kpenfound requested a review from shykes November 24, 2025 21:15
@shykes
Copy link
Contributor

shykes commented Nov 24, 2025

@kpenfound looks good, I would just call the field customizations plural, no?

Also, thinking ahead, will this allow us to also disable checks like we were discussing separately? The only tricky part I see is that argument and eg. noCheck would be mutually exclusive.

Signed-off-by: kpenfound <[email protected]>
@kpenfound kpenfound requested a review from a team as a code owner November 24, 2025 22:36
@kpenfound
Copy link
Contributor Author

@shykes yeah i was considering customizations.ignoreChecks: [] or something but I'll keep that separate so we can bikeshed and get this small change in

@kpenfound kpenfound merged commit 875f846 into dagger:main Nov 25, 2025
81 of 82 checks passed
raykao pushed a commit to raykao/dagger that referenced this pull request Dec 9, 2025
* 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]>
raykao pushed a commit to raykao/dagger that referenced this pull request Dec 14, 2025
* 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]>
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

Successfully merging this pull request may close these issues.

2 participants