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

Add descend in to list option for 'scrub_none_parameters' #261

Closed
wimnat opened this issue Feb 9, 2021 · 1 comment · Fixed by #262
Closed

Add descend in to list option for 'scrub_none_parameters' #261

wimnat opened this issue Feb 9, 2021 · 1 comment · Fixed by #262
Labels
feature This issue/PR relates to a feature request has_pr

Comments

@wimnat
Copy link
Contributor

wimnat commented Feb 9, 2021

SUMMARY

During the implementation of the scrub_none_parameters helper function we talked about whether a "descend in to lists" option was necessary. For the sake of getting it merged, the option was omitted.

Now I have sat down to start to try to use the helper function it has become clear that we definitely need the option and so this is why this issue is being raised.

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

scrub_none_parameters helper function

ADDITIONAL INFORMATION

As scrub_none_parameters has not yet been implemented anywhere I am going to make the change to descend in to lists y default.

@ansibullbot
Copy link

Files identified in the description:
None

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request has_pr
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants