Apply suggestions from code review

Thanks felixfontein for the thorough review. :-)

Co-authored-by: Felix Fontein <felix@fontein.de>

plugins/modules/firewalld.py

@@ -86,7 +86,7 @@ options:
     description:
       - Whether to apply this change to the permanent firewalld configuration.
       - As of Ansible 2.3, permanent operations can operate on firewalld configs when it is not running (requires firewalld >= 0.3.9).
-      - Note that if this is V(false), O(immediate=true).
+      - Note that if this is V(false), O(immediate=true) by default.
     type: bool
     default: false
   immediate:

plugins/modules/mount.py

@@ -24,20 +24,20 @@ options:
   path:
     description:
       - Path to the mount point (e.g. C(/mnt/files)).
-      - Before Ansible 2.3 this option was only usable as O(ignore:dest), O(ignore:destfile) and O(name).
+      - Before Ansible 2.3 this option was only usable as O(ignore:dest), O(ignore:destfile), and O(name).
     type: path
     required: true
     aliases: [ name ]
   src:
     description:
       - Device (or NFS volume, or something else) to be mounted on I(path).
-      - Required when O(state) set to V(present), V(mounted) or V(ephemeral).
+      - Required when O(state) set to V(present), V(mounted), or V(ephemeral).
       - Ignored when O(state) set to V(absent) or V(unmounted).
     type: path
   fstype:
     description:
       - Filesystem type.
-      - Required when O(state) is V(present), V(mounted) or V(ephemeral).
+      - Required when O(state) is V(present), V(mounted), or V(ephemeral).
     type: str
   opts:
     description:
@@ -71,7 +71,7 @@ options:
         I(fstab) and does not trigger or require a mount.
       - V(ephemeral) only specifies that the device is to be mounted, without changing
         I(fstab). If it is already mounted, a remount will be triggered.
-        This will always return RV(ignore:changed=True). If the mount point O(path)
+        This will always return RV(ignore:changed=true). If the mount point O(path)
         has already a device mounted on, and its source is different than O(src),
         the module will fail to avoid unexpected unmount or mount point override.
         If the mount point is not present, the mount point will be created.

plugins/modules/synchronize.py

@@ -52,7 +52,7 @@ options:
     default: push
   archive:
     description:
-      - Mirrors the rsync archive flag, enables recursive, links, perms, times, owner, group flags and C(-D).
+      - Mirrors the rsync archive flag, enables recursive, links, perms, times, owner, group flags, and C(-D).
     type: bool
     default: true
   checksum:
@@ -207,8 +207,8 @@ options:
 
 notes:
    - C(rsync) must be installed on both the local and remote host.
-   - For the M(ansible.posix.synchronize) module, the "local host" is the host `the synchronize task originates on`,
+   - For the M(ansible.posix.synchronize) module, the "local host" is the host I(the synchronize task originates on),
-     and the "destination host" is the host `synchronize is connecting to`.
+     and the "destination host" is the host I(synchronize is connecting to).
    - The "local host" can be changed to a different host by using C(delegate_to).
      This enables copying between two remote hosts or entirely on one remote machine.
    - >

plugins/modules/sysctl.py

@@ -43,7 +43,7 @@ options:
         description:
             - If V(true), performs a C(/sbin/sysctl -p) if the O(sysctl_file) is
               updated. If V(false), does not reload C(sysctl) even if the
-              C(sysctl_file) is updated.
+              O(sysctl_file) is updated.
         type: bool
         default: true
     sysctl_file: