I’ve conducted a first review and approved the collection for inclusion.
Could anybody from the @SteeringCommittee conduct the second short review as specified in the comment? i.e. to check only the following would be enough (just copy this to a comment in the discussion and go through it):
- follows the Ansible documentation standards and the style guide
- follows development conventions; as well as these other requirements:
- modules satisfy the concept of idempotency
- modules that only gather information are named
<something>_info - modules that return
ansible_factsare named<something>_factsand do not return non-facts - other modules must not allow querying information using specific
stateoption values, or similar mechanisms (likestate=getorstate=query). These features should be moved to<something>_infoor<something>_factmodules. -
check_modeis supported in all_infoand_factsmodules
Thanks