mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
version_added is required for new modules/options
This commit is contained in:
parent
718f431466
commit
9a9268de0f
1 changed files with 2 additions and 2 deletions
|
@ -643,8 +643,8 @@ The following checklist items are important guidelines for people who want to c
|
||||||
* If `required` is false/missing, `default` may be specified (assumed 'null' if missing). Ensure that the default parameter in docs matches default parameter in code.
|
* If `required` is false/missing, `default` may be specified (assumed 'null' if missing). Ensure that the default parameter in docs matches default parameter in code.
|
||||||
* Documenting `default` is not needed for `required: true`.
|
* Documenting `default` is not needed for `required: true`.
|
||||||
* Remove unnecessary doc like `aliases: []` or `choices: []`.
|
* Remove unnecessary doc like `aliases: []` or `choices: []`.
|
||||||
* Do not create choices that can be confused for booleans. choices: ['no', 'verify', 'always], 'no' will be seen as a boolean, check basic.py for BOOLEANS_* constantsto see the full lists. If your option IS a boolean, just use `type=bool`, there is no need to populate 'choices'.
|
* Do not create choices that can be confused for booleans. choices: ['no', 'verify', 'always], 'no' will be seen as a boolean, check basic.py for BOOLEANS_* constants to see the full lists. If your option IS a boolean, just use `type=bool`, there is no need to populate 'choices'.
|
||||||
* The version is not a float number and value the current development version.
|
* For new modules or options in a module add version_added. The version match the value the current development version, it is a string, not a float, so quote it.
|
||||||
* Verify that arguments in doc and module spec dict are identical.
|
* Verify that arguments in doc and module spec dict are identical.
|
||||||
* For password / secret arguments no_log=True should be set.
|
* For password / secret arguments no_log=True should be set.
|
||||||
* Requirements should be documented, using the `requirements=[]` field.
|
* Requirements should be documented, using the `requirements=[]` field.
|
||||||
|
|
Loading…
Reference in a new issue