Skip to content

Validate some markup in documentation in validate-modules - #76262

Merged
s-hertel merged 8 commits into
ansible:develfrom
felixfontein:validate-modules-docs
Nov 11, 2021
Merged

Validate some markup in documentation in validate-modules#76262
s-hertel merged 8 commits into
ansible:develfrom
felixfontein:validate-modules-docs

Conversation

@felixfontein

Copy link
Copy Markdown
Contributor
SUMMARY

Prevent M(...) from not containing a FQCN, etc.

ISSUE TYPE
  • Test Pull Request
COMPONENT NAME

validate-modules

@ansibot ansibot added WIP affects_2.13 needs_triage Needs a first human triage before being processed. test This PR relates to tests. labels Nov 9, 2021
@ansibot

ansibot commented Nov 9, 2021

Copy link
Copy Markdown
Contributor

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

test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/schema.py:121:8: unreachable: Unreachable code

click here for bot help

@ansibot ansibot added the ci_verified Changes made in this PR are causing tests to fail. label Nov 9, 2021
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Nov 9, 2021
@felixfontein felixfontein changed the title [WIP] Validate some markup in documentation in validate-modules Validate some markup in documentation in validate-modules Nov 9, 2021
@felixfontein

Copy link
Copy Markdown
Contributor Author

ready_for_review

@ansibot ansibot added core_review and removed WIP labels Nov 9, 2021
ansible-zuul Bot pushed a commit to ansible-collections/amazon.aws that referenced this pull request Nov 9, 2021
Fix docs issues.

SUMMARY
Fix some docs issues exhibited by ansible/ansible#76262.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
some modules

Reviewed-by: Brian Scholer <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: None <None>
ansible-zuul Bot pushed a commit to ansible-collections/community.aws that referenced this pull request Nov 10, 2021
Fix docs issues.

SUMMARY
Fix some docs issues exhibited by ansible/ansible#76262.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
some modules

Reviewed-by: Brian Scholer <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: None <None>
@ansibot ansibot added needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. core_review and removed core_review needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Nov 10, 2021
@russoz

russoz commented Nov 11, 2021

Copy link
Copy Markdown
Contributor

Looking better to me, but to be fair I am not familiar with that code - nor likely to become in the near future.

@s-hertel
s-hertel self-requested a review November 11, 2021 15:31
Comment thread docs/docsite/rst/dev_guide/testing_validate-modules.rst
@ansibot ansibot added docs This issue/PR relates to or includes documentation. docsite labels Nov 11, 2021
@s-hertel
s-hertel merged commit fe77bc9 into ansible:devel Nov 11, 2021
@s-hertel

Copy link
Copy Markdown
Contributor

@felixfontein Thanks for the PR!

@felixfontein
felixfontein deleted the validate-modules-docs branch November 11, 2021 18:49
@felixfontein

Copy link
Copy Markdown
Contributor Author

@briantist @russoz @s-hertel thanks a lot for reviewing and merging! :)

@jborean93 jborean93 removed the needs_triage Needs a first human triage before being processed. label Nov 11, 2021
@ansible ansible locked and limited conversation to collaborators Dec 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

affects_2.13 docs This issue/PR relates to or includes documentation. has_issue test This PR relates to tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants