Skip to content

make gettting options with falback simpler#82547

Draft
bcoca wants to merge 3 commits into
ansible:develfrom
bcoca:moar_options
Draft

make gettting options with falback simpler#82547
bcoca wants to merge 3 commits into
ansible:develfrom
bcoca:moar_options

Conversation

@bcoca

@bcoca bcoca commented Jan 16, 2024

Copy link
Copy Markdown
Member
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. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Jan 16, 2024
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Jan 17, 2024
Akasurde
Akasurde reviewed Jan 18, 2024
View reviewed changes
Comment thread lib/ansible/plugins/__init__.py Outdated
@jborean93 jborean93 removed the needs_triage Needs a first human triage before being processed. label Jan 18, 2024
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Jan 19, 2024
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Jan 19, 2024
webknjaz
webknjaz reviewed Jan 19, 2024
View reviewed changes
Comment thread lib/ansible/plugins/__init__.py Outdated
@bcoca bcoca mentioned this pull request Jan 19, 2024
Merged
@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 Feb 3, 2024
@ansibot ansibot added the stale_pr This PR has not been pushed to for more than one year. label Jan 28, 2025
@ansibot ansibot removed the stale_pr This PR has not been pushed to for more than one year. label Feb 4, 2025
Akasurde
Akasurde reviewed Feb 4, 2025
View reviewed changes
Comment thread lib/ansible/plugins/__init__.py Outdated
@bcoca @Akasurde
Update lib/ansible/plugins/__init__.py
f7a51c0 
Co-authored-by: Abhijeet Kasurde <akasurde@redhat.com>
@ansibot ansibot removed ci_verified Changes made in this PR are causing tests to fail. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. labels Feb 4, 2025
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Feb 4, 2025
@ansible ansible deleted a comment from ansibot Feb 4, 2025
@ansible ansible deleted a comment from ansibot Feb 4, 2025
@ansible ansible deleted a comment from ansibot Feb 4, 2025
@ansibot

ansibot commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

The test ansible-test sanity --test pylint [explain] failed with 1 error:

lib/ansible/plugins/__init__.py:100:4: bad-docstring-quotes: Bad docstring quotes in method, expected """, given '''

click here for bot help

@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 Feb 18, 2025
@ansibot ansibot added the stale_pr This PR has not been pushed to for more than one year. label Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@webknjaz webknjaz webknjaz left review comments

@Akasurde Akasurde Akasurde left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

ci_verified Changes made in this PR are causing tests to fail. feature This issue/PR relates to a feature request. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. stale_pr This PR has not been pushed to for more than one year.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bcoca @ansibot @webknjaz @Akasurde @jborean93