-
Notifications
You must be signed in to change notification settings - Fork 854
Where to put inputs params? #1706
Copy link
Copy link
Closed
Description
What would you rather have?
- Keep a top level
paramsin the plan as a convention; - Just put your params anywhere you like, inside
actions.
Option 1
#Plan: {
// Receive runtime parameters
- inputs: params: [name=string]: _
+ params: [name=string]: _
actions: { ... }
}dagger.#Plan & {
params: app: string
actions: { ... }
}Option 2
#Plan: {
// Receive runtime parameters
- inputs: params: [name=string]: _
actions: { ... }
}dagger.#Plan & {
actions: {
params: app: string
...
}
}
@shykes WDYT?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels