Revert whitespace changes

Author: kopytjuk

scripts/check_for_inconsistent_pandas_namespace.py

@@ -30,7 +30,8 @@
 from typing import NamedTuple
 
 ERROR_MESSAGE = (
-    "{path}:{lineno}:{col_offset}: Found both '{prefix}.{name}' and '{name}' in {path}"
+    "{path}:{lineno}:{col_offset}: "
+    "Found both '{prefix}.{name}' and '{name}' in {path}"
 )
 
 

scripts/check_test_naming.py

@@ -8,7 +8,6 @@
 NOTE: if this finds a false positive, you can add the comment `# not a test` to the
 class or function definition. Though hopefully that shouldn't be necessary.
 """
-
 from __future__ import annotations
 
 import argparse

scripts/generate_pip_deps_from_conda.py

@@ -12,7 +12,6 @@
     generated with this script:
     $ python scripts/generate_pip_deps_from_conda.py --compare
 """
-
 import argparse
 import pathlib
 import re

scripts/pandas_errors_documented.py

@@ -6,7 +6,6 @@
 
     pre-commit run pandas-errors-documented --all-files
 """
-
 from __future__ import annotations
 
 import argparse

scripts/sort_whatsnew_note.py

@@ -23,7 +23,6 @@
 
     pre-commit run sort-whatsnew-items --all-files
 """
-
 from __future__ import annotations
 
 import argparse

scripts/tests/test_check_test_naming.py

@@ -24,7 +24,10 @@
             0,
         ),
         (
-            "class Foo:  # not a test\n    pass\ndef test_foo():\n    Class.foo()\n",
+            "class Foo:  # not a test\n"
+            "    pass\n"
+            "def test_foo():\n"
+            "    Class.foo()\n",
             "",
             0,
         ),

scripts/tests/test_inconsistent_namespace_check.py

@@ -5,10 +5,14 @@
 )
 
 BAD_FILE_0 = (
-    "from pandas import Categorical\ncat_0 = Categorical()\ncat_1 = pd.Categorical()"
+    "from pandas import Categorical\n"
+    "cat_0 = Categorical()\n"
+    "cat_1 = pd.Categorical()"
 )
 BAD_FILE_1 = (
-    "from pandas import Categorical\ncat_0 = pd.Categorical()\ncat_1 = Categorical()"
+    "from pandas import Categorical\n"
+    "cat_0 = pd.Categorical()\n"
+    "cat_1 = Categorical()"
 )
 BAD_FILE_2 = (
     "from pandas import Categorical\n"

scripts/tests/test_validate_docstrings.py

@@ -34,7 +34,8 @@ def redundant_import(self, paramx=None, paramy=None) -> None:
         --------
         >>> import numpy as np
         >>> import pandas as pd
-        >>> df = pd.DataFrame(np.ones((3, 3)), columns=("a", "b", "c"))
+        >>> df = pd.DataFrame(np.ones((3, 3)),
+        ...                   columns=('a', 'b', 'c'))
         >>> df.all(axis=1)
         0    True
         1    True
@@ -49,14 +50,14 @@ def unused_import(self) -> None:
         Examples
         --------
         >>> import pandas as pdf
-        >>> df = pd.DataFrame(np.ones((3, 3)), columns=("a", "b", "c"))
+        >>> df = pd.DataFrame(np.ones((3, 3)), columns=('a', 'b', 'c'))
         """
 
     def missing_whitespace_around_arithmetic_operator(self) -> None:
         """
         Examples
         --------
-        >>> 2 + 5
+        >>> 2+5
         7
         """
 
@@ -65,14 +66,14 @@ def indentation_is_not_a_multiple_of_four(self) -> None:
         Examples
         --------
         >>> if 2 + 5:
-        ...     pass
+        ...   pass
         """
 
     def missing_whitespace_after_comma(self) -> None:
         """
         Examples
         --------
-        >>> df = pd.DataFrame(np.ones((3, 3)), columns=("a", "b", "c"))
+        >>> df = pd.DataFrame(np.ones((3,3)),columns=('a','b', 'c'))
         """
 
     def write_array_like_with_hyphen_not_underscore(self) -> None:
@@ -226,13 +227,13 @@ def test_validate_all_ignore_errors(self, monkeypatch):
                 "errors": [
                     ("ER01", "err desc"),
                     ("ER02", "err desc"),
-                    ("ER03", "err desc"),
+                    ("ER03", "err desc")
                 ],
                 "warnings": [],
                 "examples_errors": "",
                 "deprecated": True,
                 "file": "file1",
-                "file_line": "file_line1",
+                "file_line": "file_line1"
             },
         )
         monkeypatch.setattr(
@@ -271,13 +272,14 @@ def test_validate_all_ignore_errors(self, monkeypatch):
                 None: {"ER03"},
                 "pandas.DataFrame.align": {"ER01"},
                 # ignoring an error that is not requested should be of no effect
-                "pandas.Index.all": {"ER03"},
-            },
+                "pandas.Index.all": {"ER03"}
+            }
         )
         # two functions * two not global ignored errors - one function ignored error
         assert exit_status == 2 * 2 - 1
 
 
+
 class TestApiItems:
     @property
     def api_doc(self):

scripts/validate_docstrings.py

@@ -13,7 +13,6 @@
     $ ./validate_docstrings.py
     $ ./validate_docstrings.py pandas.DataFrame.head
 """
-
 from __future__ import annotations
 
 import argparse
@@ -70,10 +69,8 @@
 }
 ALL_ERRORS = set(NUMPYDOC_ERROR_MSGS).union(set(ERROR_MSGS))
 duplicated_errors = set(NUMPYDOC_ERROR_MSGS).intersection(set(ERROR_MSGS))
-assert not duplicated_errors, (
-    f"Errors {duplicated_errors} exist in both pandas "
-    "and numpydoc, should they be removed from pandas?"
-)
+assert not duplicated_errors, (f"Errors {duplicated_errors} exist in both pandas "
+                               "and numpydoc, should they be removed from pandas?")
 
 
 def pandas_error(code, **kwargs):
@@ -260,7 +257,7 @@ def pandas_validate(func_name: str):
             pandas_error(
                 "SA05",
                 reference_name=rel_name,
-                right_reference=rel_name[len("pandas.") :],
+                right_reference=rel_name[len("pandas."):],
             )
             for rel_name in doc.see_also
             if rel_name.startswith("pandas.")
@@ -368,18 +365,17 @@ def print_validate_all_results(
     for func_name, res in result.items():
         error_messages = dict(res["errors"])
         actual_failures = set(error_messages)
-        expected_failures = ignore_errors.get(func_name, set()) | ignore_errors.get(
-            None, set()
-        )
+        expected_failures = (ignore_errors.get(func_name, set())
+                             | ignore_errors.get(None, set()))
         for err_code in actual_failures - expected_failures:
             sys.stdout.write(
-                f"{prefix}{res['file']}:{res['file_line']}:"
-                f"{err_code}:{func_name}:{error_messages[err_code]}\n"
+                f'{prefix}{res["file"]}:{res["file_line"]}:'
+                f'{err_code}:{func_name}:{error_messages[err_code]}\n'
             )
             exit_status += 1
         for err_code in ignore_errors.get(func_name, set()) - actual_failures:
             sys.stdout.write(
-                f"{prefix}{res['file']}:{res['file_line']}:"
+                f'{prefix}{res["file"]}:{res["file_line"]}:'
                 f"{err_code}:{func_name}:"
                 "EXPECTED TO FAIL, BUT NOT FAILING\n"
             )
@@ -388,9 +384,8 @@ def print_validate_all_results(
     return exit_status
 
 
-def print_validate_one_results(
-    func_name: str, ignore_errors: dict[str, set[str]]
-) -> int:
+def print_validate_one_results(func_name: str,
+                               ignore_errors: dict[str, set[str]]) -> int:
     def header(title, width=80, char="#") -> str:
         full_line = char * width
         side_len = (width - len(title) - 2) // 2
@@ -401,18 +396,15 @@ def header(title, width=80, char="#") -> str:
 
     result = pandas_validate(func_name)
 
-    result["errors"] = [
-        (code, message)
-        for code, message in result["errors"]
-        if code not in ignore_errors.get(None, set())
-    ]
+    result["errors"] = [(code, message) for code, message in result["errors"]
+                        if code not in ignore_errors.get(None, set())]
 
     sys.stderr.write(header(f"Docstring ({func_name})"))
     sys.stderr.write(f"{result['docstring']}\n")
 
     sys.stderr.write(header("Validation"))
     if result["errors"]:
-        sys.stderr.write(f"{len(result['errors'])} Errors found for `{func_name}`:\n")
+        sys.stderr.write(f'{len(result["errors"])} Errors found for `{func_name}`:\n')
         for err_code, err_desc in result["errors"]:
             sys.stderr.write(f"\t{err_code}\t{err_desc}\n")
     else:
@@ -439,16 +431,14 @@ def _format_ignore_errors(raw_ignore_errors):
                     raise ValueError(
                         f"Object `{obj_name}` is present in more than one "
                         "--ignore_errors argument. Please use it once and specify "
-                        "the errors separated by commas."
-                    )
+                        "the errors separated by commas.")
                 ignore_errors[obj_name] = set(error_codes.split(","))
 
                 unknown_errors = ignore_errors[obj_name] - ALL_ERRORS
                 if unknown_errors:
                     raise ValueError(
                         f"Object `{obj_name}` is ignoring errors {unknown_errors} "
-                        f"which are not known. Known errors are: {ALL_ERRORS}"
-                    )
+                        f"which are not known. Known errors are: {ALL_ERRORS}")
 
             # global errors "PR02,ES01"
             else:
@@ -458,19 +448,27 @@ def _format_ignore_errors(raw_ignore_errors):
         if unknown_errors:
             raise ValueError(
                 f"Unknown errors {unknown_errors} specified using --ignore_errors "
-                "Known errors are: {ALL_ERRORS}"
-            )
+                "Known errors are: {ALL_ERRORS}")
 
     return ignore_errors
 
 
-def main(func_name, output_format, prefix, ignore_deprecated, ignore_errors):
+def main(
+    func_name,
+    output_format,
+    prefix,
+    ignore_deprecated,
+    ignore_errors
+):
     """
     Main entry point. Call the validation for one or for all docstrings.
     """
     if func_name is None:
         return print_validate_all_results(
-            output_format, prefix, ignore_deprecated, ignore_errors
+            output_format,
+            prefix,
+            ignore_deprecated,
+            ignore_errors
         )
     else:
         return print_validate_one_results(func_name, ignore_errors)
@@ -526,11 +524,10 @@ def main(func_name, output_format, prefix, ignore_deprecated, ignore_errors):
     args = argparser.parse_args(sys.argv[1:])
 
     sys.exit(
-        main(
-            args.function,
-            args.format,
-            args.prefix,
-            args.ignore_deprecated,
-            _format_ignore_errors(args.ignore_errors),
-        )
+        main(args.function,
+             args.format,
+             args.prefix,
+             args.ignore_deprecated,
+             _format_ignore_errors(args.ignore_errors),
+             )
     )

scripts/validate_exception_location.py

@@ -18,7 +18,6 @@
 As a pre-commit hook:
     pre-commit run validate-errors-locations --all-files
 """
-
 from __future__ import annotations
 
 import argparse