fix!: remove old -p/--parseable option

Removes outdated -p/--parseable option which duplicated
`--format=pep8` more generic option.

Author: ssbarnea

.ansible-lint

@@ -13,7 +13,6 @@ exclude_paths:
   - .cache/ # implicit unless exclude_paths is defined in config
   - test/fixtures/formatting-before/
   - test/fixtures/formatting-prettier/
-# parseable: true
 # quiet: true
 # strict: true
 # verbosity: 1

.config/dictionary.txt

@@ -46,7 +46,6 @@ nodeps
 nomatchestest
 nxos
 octals
-parseable
 pathex
 pbrun
 pfexec

.pre-commit-config.yaml

@@ -30,7 +30,7 @@ repos:
     hooks:
       - id: check-useless-excludes
   - repo: https://github.com/astral-sh/uv-pre-commit
-    rev: 0.9.18
+    rev: 0.9.21
     hooks:
       - id: uv-sync
       - id: uv-lock
@@ -146,7 +146,7 @@ repos:
         entry: yamllint --strict
 
   - repo: https://github.com/tombi-toml/tombi-pre-commit
-    rev: v0.7.11
+    rev: v0.7.14
     hooks:
       - id: tombi-format
         alias: toml

docs/usage.md

@@ -17,7 +17,7 @@ Ansible-lint prints output on both `stdout` and `stderr`.
 - `stderr` displays logging and free-form messages like statistics.
 
 Most `ansible-lint` examples use pep8 as the output format (`-p`) which is
-machine parseable.
+machine parsable.
 
 Ansible-lint also print errors using their [annotation] format when it detects
 the `GITHUB_ACTIONS=true` and `GITHUB_WORKFLOW=...` variables.

src/ansiblelint/app.py

@@ -372,7 +372,7 @@ def choose_formatter_factory(
         r = formatters.CodeclimateJSONFormatter
     elif options_list.format == "sarif":
         r = formatters.SarifFormatter
-    elif options_list.parseable or options_list.format == "pep8":
+    elif options_list.format == "pep8":
         r = formatters.ParseableFormatter
     return r
 

src/ansiblelint/cli.py

@@ -317,14 +317,6 @@ def get_cli_parser() -> argparse.ArgumentParser:
         choices=PROFILES.keys(),
         help="Specify which rules profile to be used.",
     )
-    parser.add_argument(
-        "-p",
-        "--parseable",
-        dest="parseable",
-        default=False,
-        action="store_true",
-        help="parseable output, same as '-f pep8'",
-    )
     parser.add_argument(
         "--project-dir",
         dest="project_dir",
@@ -493,7 +485,6 @@ def merge_config(file_config: dict[Any, Any], cli_config: Options) -> Options:
     """Combine the file config with the CLI args."""
     bools = (
         "display_relative_path",
-        "parseable",
         "quiet",
         "strict",
         "use_default_rules",

src/ansiblelint/config.py

@@ -143,7 +143,6 @@ class Options:  # pylint: disable=too-many-instance-attributes
     list_tags: bool = False
     write_list: list[str] = field(default_factory=list)
     write_exclude_list: list[str] = field(default_factory=list)
-    parseable: bool = False
     quiet: bool = False
     rulesdirs: list[Path] = field(default_factory=list)
     skip_list: list[str] = field(default_factory=list)

src/ansiblelint/schemas/ansible-lint-config.json

@@ -101,11 +101,6 @@
       "title": "Only Builtins Allow Modules",
       "type": "array"
     },
-    "parseable": {
-      "default": true,
-      "title": "Parseable",
-      "type": "boolean"
-    },
     "profile": {
       "enum": [
         "min",

test/fixtures/parseable.yml

@@ -23,7 +23,6 @@ def fixture_base_arguments() -> list[str]:
 @pytest.mark.parametrize(
     ("args", "config_path"),
     (
-        pytest.param(["-p"], "test/fixtures/parseable.yml", id="1"),
         pytest.param(["-q"], "test/fixtures/quiet.yml", id="2"),
         pytest.param(
             ["-r", "test/fixtures/rules/"],