refactor: Remove deprecated code

PR-433: #433
Co-authored-by: Timothée Mazzucotelli <dev@pawamoy.fr>

Author: johnslavik

docs/guide/contributors/workflow.md

@@ -57,10 +57,6 @@ Deprecated code should also be marked as legacy code. We use [Yore](https://pawa
 Examples:
 
 ```python title="Remove function when we bump to 2.0"
-# YORE: Bump 2: Remove block.
-def deprecated_function():
-    ...
-```
 
 ```python title="Simplify imports when Python 3.15 is EOL"
 # YORE: EOL 3.15: Replace block with line 4.

docs/reference/api/agents.md

@@ -70,8 +70,4 @@ Griffe is able to analyze code both statically and dynamically.
 
 ::: griffe.safe_get_value
 
-<!-- YORE: Bump 2: Remove line. -->
-## **Deprecated API**
 
-<!-- YORE: Bump 2: Remove line. -->
-::: griffe.ExportedName

docs/reference/api/git.md

@@ -291,8 +291,6 @@ def check_api(ctx: Context, *cli_args: str) -> None:
             color=True,
             extensions=[
                 "griffe_inherited_docstrings",
-                # YORE: Bump 2: Remove line.
-                "scripts/griffe_exts.py",
                 "unpack_typeddict",
             ],
         ).add_args(*cli_args),

duties.py

@@ -110,8 +110,6 @@ nav:
       - Parsers: reference/api/docstrings/parsers.md
     - Exceptions: reference/api/exceptions.md
     - Expressions: reference/api/expressions.md
-    # YORE: Bump 2: Remove line.
-    - Git utilities: reference/api/git.md
     - Loggers: reference/api/loggers.md
     - Helpers: reference/api/helpers.md
 - Changelog: changelog.md
@@ -231,8 +229,6 @@ plugins:
           docstring_section_style: list
           extensions:
           - griffe_inherited_docstrings
-          # YORE: Bump 2: Remove line.
-          - scripts/griffe_exts.py
           - unpack_typeddict
           heading_level: 2
           inherited_members: true

mkdocs.yml

@@ -164,9 +164,6 @@
 
 from __future__ import annotations
 
-import warnings
-from typing import Any
-
 from griffe._internal.agents.inspector import Inspector, inspect
 from griffe._internal.agents.nodes.assignments import get_instance_names, get_name, get_names
 from griffe._internal.agents.nodes.ast import (
@@ -181,9 +178,7 @@
     ast_siblings,
 )
 from griffe._internal.agents.nodes.docstrings import get_docstring
-
-# YORE: Bump 2: Replace `ExportedName, ` with `` within line.
-from griffe._internal.agents.nodes.exports import ExportedName, get__all__, safe_get__all__
+from griffe._internal.agents.nodes.exports import get__all__, safe_get__all__
 from griffe._internal.agents.nodes.imports import relative_to_absolute
 from griffe._internal.agents.nodes.parameters import ParametersType, get_parameters
 from griffe._internal.agents.nodes.runtime import ObjectNode
@@ -383,33 +378,9 @@
     vtree,
 )
 
-# YORE: Bump 2: Remove block.
-_deprecated_names = (
-    "assert_git_repo",
-    "get_latest_tag",
-    "get_repo_root",
-    "tmp_worktree",
-)
-
-
-# YORE: Bump 2: Remove block.
-def __getattr__(name: str) -> Any:
-    if name in _deprecated_names:
-        from griffe._internal import git  # noqa: PLC0415
-
-        warnings.warn(
-            f"The `{name}` function is deprecated and will become unavailable in the next major version.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
-        return getattr(git, f"_{name}")
-
-    raise AttributeError(f"module '{__name__}' has no attribute '{name}'")
-
-
 # Regenerate this list with the following Python snippet:
 # import griffe
-# names = sorted(n for n in dir(griffe) if not n.startswith("_") and n not in ("Any", "annotations", "lazy_importing", "warnings"))
+# names = sorted(n for n in dir(griffe) if not n.startswith("_") and n not in ("annotations",))
 # print('__all__ = [\n    "' + '",\n    "'.join(names) + '",\n]')
 __all__ = [
     "DEFAULT_LOG_LEVEL",
@@ -468,8 +439,6 @@ def __getattr__(name: str) -> Any:
     "DocstringWarn",
     "DocstringYield",
     "ExplanationStyle",
-    # YORE: Bump 2: Remove line.
-    "ExportedName",
     "Expr",
     "ExprAttribute",
     "ExprBinOp",
@@ -565,7 +534,6 @@ def __getattr__(name: str) -> Any:
     "UnimportableModuleError",
     "UnpackTypedDictExtension",
     "Visitor",
-    "assert_git_repo",
     "ast_children",
     "ast_first_child",
     "ast_kind",
@@ -591,13 +559,11 @@ def __getattr__(name: str) -> Any:
     "get_docstring",
     "get_expression",
     "get_instance_names",
-    "get_latest_tag",
     "get_logger",
     "get_name",
     "get_names",
     "get_parameters",
     "get_parser",
-    "get_repo_root",
     "get_value",
     "htree",
     "infer_docstring_style",
@@ -635,7 +601,6 @@ def __getattr__(name: str) -> Any:
     "temporary_pypackage",
     "temporary_visited_module",
     "temporary_visited_package",
-    "tmp_worktree",
     "typing_overload",
     "visit",
     "vtree",

scripts/griffe_exts.py

@@ -4,7 +4,6 @@
 
 import ast
 from contextlib import suppress
-from dataclasses import dataclass
 from typing import TYPE_CHECKING, Any, Callable
 
 from griffe._internal.agents.nodes.values import get_value
@@ -16,20 +15,6 @@
     from griffe._internal.models import Module
 
 
-# YORE: Bump 2: Remove block.
-@dataclass
-class ExportedName:
-    """Deprecated. An intermediate class to store names.
-
-    The [`get__all__`][griffe.get__all__] function now returns instances of [`ExprName`][griffe.ExprName] instead.
-    """
-
-    name: str
-    """The exported name."""
-    parent: Module
-    """The parent module."""
-
-
 def _extract_attribute(node: ast.Attribute, parent: Module) -> list[str | ExprName]:
     return [ExprName(name=node.attr, parent=_extract(node.value, parent)[0])]
 

src/griffe/__init__.py

@@ -3,8 +3,7 @@
 from __future__ import annotations
 
 import re
-from typing import TYPE_CHECKING, Any, Literal, TypedDict
-from warnings import warn
+from typing import TYPE_CHECKING, Literal, TypedDict
 
 from griffe._internal.enumerations import Parser
 
@@ -121,8 +120,6 @@ def infer_docstring_style(
     style_order: list[Parser] | list[DocstringStyle] | None = None,
     default: Parser | DocstringStyle | None = None,
     per_style_options: PerStyleOptions | None = None,
-    # YORE: Bump 2: Remove line.
-    **options: Any,
 ) -> tuple[Parser | None, list[DocstringSection] | None]:
     """Infer the parser to use for the docstring.
 
@@ -142,21 +139,13 @@ def infer_docstring_style(
         style_order: The order of the styles to try when inferring the parser.
         default: The default parser to use if the inference fails.
         per_style_options: Additional parsing options per style.
-        **options: Deprecated. Use `per_style_options` instead.
 
     Returns:
         The inferred parser, and optionally parsed sections (when method is 'max_sections').
     """
     from griffe._internal.docstrings.parsers import parsers  # noqa: PLC0415
 
-    # YORE: Bump 2: Replace block with `per_style_options = per_style_options or {}`.
-    if options:
-        if per_style_options:
-            raise ValueError("Cannot use both `options` and `per_style_options`.")
-        warn("`**options` is deprecated. Use `per_style_options` instead.", DeprecationWarning, stacklevel=2)
-        per_style_options = {"google": options, "numpy": options, "sphinx": options}  # type: ignore[assignment]
-    elif not per_style_options:
-        per_style_options = {}
+    per_style_options = per_style_options or {}
 
     style_order = [Parser(style) if isinstance(style, str) else style for style in style_order or _default_style_order]
 
@@ -203,8 +192,6 @@ def parse_auto(
     style_order: list[Parser] | list[DocstringStyle] | None = None,
     default: Parser | DocstringStyle | None = None,
     per_style_options: PerStyleOptions | None = None,
-    # YORE: Bump 2: Remove line.
-    **options: Any,
 ) -> list[DocstringSection]:
     """Parse a docstring by automatically detecting the style it uses.
 
@@ -217,21 +204,13 @@ def parse_auto(
         style_order: The order of the styles to try when inferring the parser.
         default: The default parser to use if the inference fails.
         per_style_options: Additional parsing options per style.
-        **options: Deprecated. Use `per_style_options` instead.
 
     Returns:
         A list of docstring sections.
     """
     from griffe._internal.docstrings.parsers import parse  # noqa: PLC0415
 
-    # YORE: Bump 2: Replace block with `per_style_options = per_style_options or {}`.
-    if options:
-        if per_style_options:
-            raise ValueError("Cannot use both `options` and `per_style_options`.")
-        warn("`**options` are deprecated. Use `per_style_options` instead.", DeprecationWarning, stacklevel=2)
-        per_style_options = {"google": options, "numpy": options, "sphinx": options}  # type: ignore[assignment]
-    elif not per_style_options:
-        per_style_options = {}
+    per_style_options = per_style_options or {}
 
     style, sections = infer_docstring_style(
         docstring,

src/griffe/_internal/agents/nodes/exports.py

@@ -5,7 +5,6 @@
 import re
 from contextlib import suppress
 from typing import TYPE_CHECKING, TypedDict
-from warnings import warn
 
 from griffe._internal.docstrings.models import (
     DocstringAttribute,
@@ -892,8 +891,6 @@ def parse_google(
     warn_unknown_params: bool = True,
     warn_missing_types: bool = True,
     warnings: bool = True,
-    # YORE: Bump 2: Remove line.
-    **options: Any,
 ) -> list[DocstringSection]:
     """Parse a Google-style docstring.
 
@@ -921,7 +918,6 @@ def parse_google(
         warn_unknown_params: Warn about documented parameters not appearing in the signature.
         warn_missing_types: Warn about missing types/annotations for parameters, return values, etc.
         warnings: Whether to log warnings at all.
-        **options: Swallowing keyword arguments for backward-compatibility.
 
     Returns:
         A list of docstring sections.
@@ -932,10 +928,6 @@ def parse_google(
     in_code_block = False
     lines = docstring.lines
 
-    # YORE: Bump 2: Remove block.
-    if options:
-        warn("Passing additional options is deprecated, these options are ignored.", DeprecationWarning, stacklevel=2)
-
     options = {
         "ignore_init_summary": ignore_init_summary,
         "trim_doctest_flags": trim_doctest_flags,