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

[WIP] Make sure that none is not accepted for required module options #82731

Draft
wants to merge 10 commits into
base: devel
Choose a base branch
from

Conversation

s-hertel
Copy link
Contributor

SUMMARY

Rebase/port of #72248 to add a new check for None for required module/role argument spec options and deprecate treating explicitly specified None options as "not specified" implicitly by default.

Fixes #82724

ISSUE TYPE
  • Feature Pull Request

@ansibot ansibot added feature This issue/PR relates to a feature request. needs_triage Needs a first human triage before being processed. has_issue labels Feb 22, 2024
@jborean93 jborean93 removed the needs_triage Needs a first human triage before being processed. label Feb 22, 2024
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Mar 7, 2024
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_issue stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

check_required_one_of does not check for null params values
4 participants