Tidy up validate-modules ignores for packaging modules (#1289) (#1291)

* fixed validation-modules for plugins/modules/packaging/language/bower.py

(cherry picked from commit deddce02fa)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>

plugins/modules/packaging/os/apk.py

@@ -45,11 +45,11 @@ options:
   state:
     description:
       - Indicates the desired package(s) state.
-      - C(present) ensures the package(s) is/are present.
-      - C(absent) ensures the package(s) is/are absent.
+      - C(present) ensures the package(s) is/are present. C(installed) can be used as an alias.
+      - C(absent) ensures the package(s) is/are absent. C(removed) can be used as an alias.
       - C(latest) ensures the package(s) is/are present and the latest version(s).
     default: present
-    choices: [ "present", "absent", "latest" ]
+    choices: [ "present", "absent", "latest", "installed", "removed" ]
     type: str
   update_cache:
     description:

plugins/modules/packaging/os/apt_rpm.py

@@ -27,7 +27,7 @@ options:
   state:
     description:
       - Indicates the desired package state.
-    choices: [ absent, present ]
+    choices: [ absent, present, installed, removed ]
     default: present
     type: str
   update_cache:

plugins/modules/packaging/os/macports.py

@@ -35,7 +35,7 @@ options:
     state:
         description:
             - Indicates the desired state of the port.
-        choices: [ 'present', 'absent', 'active', 'inactive' ]
+        choices: [ 'present', 'absent', 'active', 'inactive', 'installed', 'removed']
         default: present
         type: str
     upgrade:

plugins/modules/packaging/os/openbsd_pkg.py

@@ -31,7 +31,7 @@ options:
           - C(present) will make sure the package is installed.
             C(latest) will make sure the latest version of the package is installed.
             C(absent) will make sure the specified package is not installed.
-        choices: [ absent, latest, present ]
+        choices: [ absent, latest, present, installed, removed ]
         default: present
         type: str
     build:

plugins/modules/packaging/os/opkg.py

@@ -27,7 +27,7 @@ options:
     state:
         description:
             - state of the package
-        choices: [ 'present', 'absent' ]
+        choices: [ 'present', 'absent', 'installed', 'removed' ]
         default: present
         type: str
     force:
@@ -45,7 +45,6 @@ options:
             - "remove"
             - "checksum"
             - "removal-of-dependent-packages"
-        default: absent
         type: str
     update_cache:
         description:

plugins/modules/packaging/os/pacman.py

@@ -32,7 +32,7 @@ options:
         description:
             - Desired state of the package.
         default: present
-        choices: [ absent, latest, present ]
+        choices: [ absent, latest, present, installed, removed ]
         type: str
 
     force:

plugins/modules/packaging/os/pkg5.py

@@ -28,7 +28,7 @@ options:
   state:
     description:
       - Whether to install (I(present), I(latest)), or remove (I(absent)) a package.
-    choices: [ absent, latest, present ]
+    choices: [ absent, latest, present, installed, removed, uninstalled ]
     default: present
     type: str
   accept_licenses:

plugins/modules/packaging/os/rhn_channel.py

@@ -32,6 +32,7 @@ options:
         description:
             - Whether the channel should be present or not, taking action if the state is different from what is stated.
         default: present
+        choices: [ present, absent ]
         type: str
     url:
         description:

plugins/modules/packaging/os/slackpkg.py

@@ -33,7 +33,7 @@ options:
     state:
         description:
             - state of the package, you can use "installed" as an alias for C(present) and removed as one for C(absent).
-        choices: [ 'present', 'absent', 'latest' ]
+        choices: [ 'present', 'absent', 'latest', 'installed', 'removed' ]
         required: false
         default: present
         type: str

plugins/modules/packaging/os/urpmi.py

@@ -27,7 +27,7 @@ options:
   state:
     description:
       - Indicates the desired package state.
-    choices: [ absent, present ]
+    choices: [ absent, present, installed, removed ]
     default: present
     type: str
   update_cache:
@@ -36,12 +36,12 @@ options:
     type: bool
     default: no
     aliases: ['update-cache']
-  no-recommends:
+  no_recommends:
     description:
       - Corresponds to the C(--no-recommends) option for I(urpmi).
     type: bool
     default: yes
-    aliases: ['no_recommends']
+    aliases: ['no-recommends']
   force:
     description:
       - Assume "yes" is the answer to any question urpmi has to ask.

plugins/modules/packaging/os/xbps.py

@@ -29,7 +29,7 @@ options:
         description:
             - Desired state of the package.
         default: "present"
-        choices: ["present", "absent", "latest"]
+        choices: ["present", "absent", "latest", "installed", "removed"]
         type: str
     recurse:
         description:

plugins/modules/packaging/os/zypper.py

@@ -49,7 +49,7 @@ options:
             C(dist-upgrade) will make sure the latest version of all installed packages from all enabled repositories is installed.
           - When using C(dist-upgrade), I(name) should be C('*').
         required: false
-        choices: [ present, latest, absent, dist-upgrade ]
+        choices: [ present, latest, absent, dist-upgrade, installed, removed ]
         default: "present"
         type: str
     type: