-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Panos/doc fragment #37696
Panos/doc fragment #37696
Conversation
The test
The test
The test
The test
The test
The test
The test
The test
The test
|
What is needed for this to be merged? |
Can you please rebase then I'll look at getting this merged. thanks |
options: | ||
ip_address: | ||
description: | ||
- IP address (or hostname) of PAN-OS device |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a fullstop at the end of this (and other)
required: true | ||
password: | ||
description: | ||
- password for authentication |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Password
required: true | ||
username: | ||
description: | ||
- username for authentication |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Username
- Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance
- Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module.
Updated some formatting errors to make ansibot happy.
Thank you for contributing to cleaner documentation! |
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
* Create PanOS module documentation fragment - Module documentation fragment currently holds 3 parameters - It most likely won't be used in every single module since there is some variance * Modified PanOS module for use doc_frags - Where documentation is consistent, module documentation now uses a documentation fragment instead of all documentation being contained in the module. * Formatting and syntax error fixes Updated some formatting errors to make ansibot happy. * Revisions for grammar
SUMMARY
Many of the PanOS modules have duplicate documentation. This pull request’s goal is to summarize documentation, where possible, in a documentation fragment instead of in the module itself.
ISSUE TYPE
COMPONENT NAME
panos_admin
This affects other modules as well
ANSIBLE VERSION
ADDITIONAL INFORMATION
Not all modules are affected by this change. If a documentation fragment should exist only when all modules in a family can use it, it likely makes sense to cancel this PR.