Skip to content
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

Pass null parameters #563

Open
shcrist opened this issue Jan 17, 2020 · 1 comment
Open

Pass null parameters #563

shcrist opened this issue Jan 17, 2020 · 1 comment

Comments

@shcrist
Copy link

shcrist commented Jan 17, 2020

Change the Exception, SkipRule, and SkipRuleType to allow for null values to be passed. Currently these values do not allow nulls. To create a configuration that accepts those values from a manifest the values must be checked in the configuration which creates a cumbersome switch statement (Eight cases) with redundant resource definitions containing the proper parameter list. If using multiple resources in a configuration it quickly becomes hundreds of lines of script. Allowing for passing a null value would allow for a single definition and move the evaluation of the parameters to the resource. In the resource the check could be part of the existing parameter validation code with a few lines of script.

@stevehose
Copy link
Contributor

Shaun, this is in the pipeline and it does need to happen.

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

No branches or pull requests

2 participants