Fix handling of -- as separator of positional arguments and flags

pylint-dev · Jul 17, 2022 · f1d27ff · f1d27ff
1 parent bf29a55
commit f1d27ff
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/doc/whatsnew/2/2.14/full.rst b/doc/whatsnew/2/2.14/full.rst
@@ -6,6 +6,9 @@ What's New in Pylint 2.14.5?
 Release date: TBA
 
 
+* Fixed handling of ``--`` as separator between positional arguments and flags.
+
+  Closes #7003
 
 What's New in Pylint 2.14.4?
 ----------------------------

diff --git a/pylint/config/config_initialization.py b/pylint/config/config_initialization.py
@@ -76,7 +76,8 @@ def _config_initialization(
     unrecognized_options: list[str] = []
     for opt in parsed_args_list:
         if opt.startswith("--"):
-            unrecognized_options.append(opt[2:])
+            if len(opt) > 2:
+                unrecognized_options.append(opt[2:])
         elif opt.startswith("-"):
             unrecognized_options.append(opt[1:])
     if unrecognized_options:

diff --git a/tests/config/test_config.py b/tests/config/test_config.py
@@ -116,3 +116,13 @@ def test_short_verbose(capsys: CaptureFixture) -> None:
     Run([str(EMPTY_MODULE), "-v"], exit=False)
     output = capsys.readouterr()
     assert "Using config file" in output.err
+
+
+def test_argument_separator(capsys: CaptureFixture) -> None:
+    """Check that we support using '--' to separate argument types.
+
+    Reported in https://github.com/PyCQA/pylint/issues/7003.
+    """
+    Run(["--", str(EMPTY_MODULE)], exit=False)
+    output = capsys.readouterr()
+    assert not output.err