-
Notifications
You must be signed in to change notification settings - Fork 13
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
Enable selecting component to execute #5
Comments
implementation detail question. I can imagine not having to specify anything if there is only one component. In cases where there are multiple, would it be required to specify the ID, or if it is not specified, we fall back to executing the first component? I think we should make it compulsory in the case of multiple components as otherwise, the behavior can be confusing if the user changes the order of the components specified. |
I agree. In the case of >1 components require the component selection flag. |
Is this the right idiom? Suppose, in my manifest, I write:
It's not very obvious what this could mean. In any other trigger, that means all those triggers are instantiated and respond to events. The equivalent semantics would be that all command components run. But then again, I am not sure what a meaningful use case is for multiple commands in a single application. Perhaps multiple instances of the |
It could potentially be useful to package multiple command into a suite (I would like to imagine something like As far as the manifest in the example provided, I would hope the author has descriptive name 😅 . |
One usecase I can imagine is not needing to duplicate variable definitions across each application on a per command basis. So multiple commands in an application can share a singular set of variables. |
(my comment is not a feature request, just exploring the usefulness of multiple components) There are two main scenarios I can think of here:
|
Tracking this so it doesn't get lost.
Currently the trigger can execute only the first
command
component. We should define a trigger flag--id/--component-id
that specifies which command component to execute.The text was updated successfully, but these errors were encountered: