-
Notifications
You must be signed in to change notification settings - Fork 44
Add missing input.name parameter for evaluation of config.name #16
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
@KowalskiTom should the role be named with the same name than the lambda? After your role PR is merged, this could contain also a role name. Otherwise seems to work as expected. |
@laardee good call. I have added this in. |
@eahefnawy any plans to merge this one? It's kind of a needed option in my humble opinion |
What's stopping this from being merged? I'd be willing to help get this thing up ASAP |
@eahefnawy we desperately need this, could you please review? |
Another one here needing this ✋ |
@eahefnawy we need this as well. |
@eahefnawy please merge |
Need this as well :) |
Thanks all! We rebuilt the entire thing from scratch and updated it for Components V2 and incorporated this change. |
@eahefnawy |
It's already out. Check out the readme: |
Add missing input.name parameter for evaluation of config.name