Removed experimental flag from multiple rules (#3037)

Author: ssbarnea

examples/playbooks/strict-mode.yml

@@ -2,6 +2,7 @@
 - name: Fixture for test_strict
   hosts: localhost
   tasks:
-    - ansible.builtin.debug: # <-- name should be first key (warning)
-        msg: "Hello World"
-      name: Display debug information
+    - name: Display debug information
+      ansible.builtin.stat:
+        path2: echo "Hello World" # <-- args[module] due to invalid use of path2 instead of path
+      changed_when: false

src/ansiblelint/config.py

@@ -27,16 +27,8 @@
 )
 
 DEFAULT_WARN_LIST = [
-    "avoid-implicit",
     "experimental",
-    "fqcn[action]",
-    "fqcn[redirect]",
     "jinja[spacing]",  # warning until we resolve all reported false-positives
-    "name[casing]",
-    "name[play]",
-    "name[prefix]",
-    "role-name",
-    "role-name[path]",  # too new
 ]
 
 DEFAULT_KINDS = [

src/ansiblelint/rules/avoid_implicit.py

@@ -21,7 +21,7 @@ class AvoidImplicitRule(AnsibleLintRule):
         "``copy`` with ``content`` to ensure correctness."
     )
     severity = "MEDIUM"
-    tags = ["unpredictability", "experimental"]
+    tags = ["unpredictability"]
     version_added = "v6.8.0"
 
     def matchtask(

src/ansiblelint/rules/ignore_errors.py

@@ -21,7 +21,7 @@ class IgnoreErrorsRule(AnsibleLintRule):
         "to reduce the risk of ignoring important failures."
     )
     severity = "LOW"
-    tags = ["unpredictability", "experimental"]
+    tags = ["unpredictability"]
     version_added = "v5.0.7"
 
     def matchtask(

src/ansiblelint/rules/key_order.py

@@ -53,7 +53,7 @@ class KeyOrderRule(AnsibleLintRule):
     id = "key-order"
     shortdesc = __doc__
     severity = "LOW"
-    tags = ["formatting", "experimental"]
+    tags = ["formatting"]
     version_added = "v6.6.2"
     needs_raw_task = True
 

src/ansiblelint/rules/no_free_form.py

@@ -19,7 +19,7 @@ class NoFreeFormRule(AnsibleLintRule):
     id = "no-free-form"
     description = "Avoid free-form inside files as it can produce subtile bugs."
     severity = "MEDIUM"
-    tags = ["syntax", "risk", "experimental"]
+    tags = ["syntax", "risk"]
     version_added = "v6.8.0"
     needs_raw_task = True
     cmd_shell_re = re.compile(

src/ansiblelint/rules/no_prompting.py

@@ -20,7 +20,7 @@ class NoPromptingRule(AnsibleLintRule):
         "Disallow the use of vars_prompt or ansible.builtin.pause to better"
         "accommodate unattended playbook runs and use in CI pipelines."
     )
-    tags = ["opt-in", "experimental"]
+    tags = ["opt-in"]
     severity = "VERY_LOW"
     version_added = "v6.0.3"
 

src/ansiblelint/rules/risky_file_permissions.py

@@ -78,7 +78,7 @@ class MissingFilePermissionsRule(AnsibleLintRule):
     )
     link = "https://github.com/ansible/ansible/issues/71200"
     severity = "VERY_HIGH"
-    tags = ["unpredictability", "experimental"]
+    tags = ["unpredictability"]
     version_added = "v4.3.0"
 
     _modules = _MODULES

src/ansiblelint/rules/run_once.py

@@ -19,7 +19,7 @@ class RunOnce(AnsibleLintRule):
     link = "https://docs.ansible.com/ansible/latest/reference_appendices/playbooks_keywords.html"
     description = "When using run_once, we should avoid using strategy as free."
 
-    tags = ["idiom", "experimental"]
+    tags = ["idiom"]
     severity = "MEDIUM"
 
     def matchplay(self, file: Lintable, data: dict[str, Any]) -> list[MatchError]:

src/ansiblelint/rules/schema.py

@@ -50,7 +50,7 @@ class ValidateSchemaRule(AnsibleLintRule):
 
     id = "schema"
     severity = "VERY_HIGH"
-    tags = ["core", "experimental"]
+    tags = ["core"]
     version_added = "v6.1.0"
 
     def matchtask(