-
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
This action should validate all the parameters it's given #35
Comments
Yea we defiantly need this. |
If you'd like to work on a PR, that'd be great. I think the easiest way to do this is to write a shell script that accepts all the parameters and validates them. I think validation rules are:
|
I ended up doing this as part of some other work. It's available in the v1.0.0-beta1 release, which I hope to mark stable in the next few weeks. |
Oh cool. I checked out project and I did not find out where should I add this feature and what should I do. Happy to hear It's fixed now. Thanks |
In #34, the issue was that
command
was set to an empty string. It'd be nice if this action validate the arguments it's given and gave a clear error if they aren't valid.The text was updated successfully, but these errors were encountered: