refactor: Update code for Griffe 0.48 (removing deprecation warnings)

Issue-173: #173

Author: pawamoy

config/pytest.ini

@@ -12,3 +12,6 @@ filterwarnings =
   error
   # TODO: remove once pytest-xdist 4 is released
   ignore:.*rsyncdir:DeprecationWarning:xdist
+  # TODO: remove once Griffe releases v1
+  ignore:.*`get_logger`:DeprecationWarning:_griffe
+  ignore:.*`name`:DeprecationWarning:_griffe

pyproject.toml

@@ -30,7 +30,7 @@ classifiers = [
 ]
 dependencies = [
     "mkdocstrings>=0.25",
-    "griffe>=0.47",
+    "griffe>=0.48",
 ]
 
 [project.urls]

src/mkdocstrings_handlers/python/handler.py

@@ -12,12 +12,15 @@
 from pathlib import Path
 from typing import TYPE_CHECKING, Any, BinaryIO, ClassVar, Iterator, Mapping, Sequence
 
-from griffe.collections import LinesCollection, ModulesCollection
-from griffe.docstrings.parsers import Parser
-from griffe.exceptions import AliasResolutionError
-from griffe.extensions import load_extensions
-from griffe.loader import GriffeLoader
-from griffe.logger import patch_loggers
+from griffe import (
+    AliasResolutionError,
+    GriffeLoader,
+    LinesCollection,
+    ModulesCollection,
+    Parser,
+    load_extensions,
+    patch_logger,
+)
 from mkdocstrings.extension import PluginError
 from mkdocstrings.handlers.base import BaseHandler, CollectionError, CollectorItem
 from mkdocstrings.inventory import Inventory
@@ -47,7 +50,7 @@ def chdir(path: str) -> Iterator[None]:  # noqa: D103
 
 logger = get_logger(__name__)
 
-patch_loggers(get_logger)
+patch_logger(get_logger)
 
 
 class PythonHandler(BaseHandler):

src/mkdocstrings_handlers/python/rendering.py

@@ -12,19 +12,20 @@
 from pathlib import Path
 from typing import TYPE_CHECKING, Any, Callable, Match, Pattern, Sequence
 
-from griffe.dataclasses import Alias, Object
-from griffe.docstrings.dataclasses import (
+from griffe import (
+    Alias,
     DocstringSectionAttributes,
     DocstringSectionClasses,
     DocstringSectionFunctions,
     DocstringSectionModules,
+    Object,
 )
 from jinja2 import TemplateNotFound, pass_context, pass_environment
 from markupsafe import Markup
 from mkdocstrings.loggers import get_logger
 
 if TYPE_CHECKING:
-    from griffe.dataclasses import Attribute, Class, Function, Module
+    from griffe import Attribute, Class, Function, Module
     from jinja2 import Environment, Template
     from jinja2.runtime import Context
     from mkdocstrings.handlers.base import CollectorItem
@@ -364,11 +365,9 @@ def _keep_object(name: str, filters: Sequence[tuple[Pattern, bool]]) -> bool:
         if regex.search(name):
             keep = not exclude
     if keep is None:
-        if rules == {False}:
-            # only included stuff, no match = reject
-            return False
-        # only excluded stuff, or included and excluded stuff, no match = keep
-        return True
+        # When we only include stuff, no match = reject.
+        # When we only exclude stuff, or include and exclude stuff, no match = keep.
+        return rules != {False}
     return keep
 
 

src/mkdocstrings_handlers/python/templates/material/_base/attribute.html.jinja

@@ -4,7 +4,7 @@ This template renders a Python attribute (or variable).
 This can be a module attribute or a class attribute.
 
 Context:
-  attribute (griffe.dataclasses.Attribute): The attribute to render.
+  attribute (griffe.Attribute): The attribute to render.
   root (bool): Whether this is the root object, injected with `:::` in a Markdown page.
   heading_level (int): The HTML heading level to use.
   config (dict): The configuration options.

src/mkdocstrings_handlers/python/templates/material/_base/children.html.jinja

@@ -4,7 +4,7 @@ This template iterates on members of a given object and renders them.
 It can group members by category (attributes, classes, functions, modules) or render them in a flat list.
 
 Context:
-  obj (griffe.dataclasses.Object): The object to render.
+  obj (griffe.Object): The object to render.
   config (dict): The configuration options.
   root_members (bool): Whether the object is the root object.
   heading_level (int): The HTML heading level to use.

src/mkdocstrings_handlers/python/templates/material/_base/class.html.jinja

@@ -3,7 +3,7 @@
 This template renders a Python class.
 
 Context:
-  class (griffe.dataclasses.Class): The class to render.
+  class (griffe.Class): The class to render.
   root (bool): Whether this is the root object, injected with `:::` in a Markdown page.
   heading_level (int): The HTML heading level to use.
   config (dict): The configuration options.

src/mkdocstrings_handlers/python/templates/material/_base/docstring.html.jinja

@@ -5,7 +5,7 @@ Griffe parses docstrings into a list of sections, each with a `kind` and a `valu
 This template can then iterate on these sections and render them according to the configuration.
 
 Context:
-  docstring_sections (list[griffe.docstrings.dataclasses.DocstringSection]): The list of docstring sections.
+  docstring_sections (list[griffe.DocstringSection]): The list of docstring sections.
   config (dict): The configuration dictionary.
   heading_level (int): The heading level to use for Markdown conversion.
   html_id (str): The HTML ID to use for Markdown conversion.

src/mkdocstrings_handlers/python/templates/material/_base/docstring/admonition.html.jinja

@@ -3,7 +3,7 @@
 This template renders admonitions using the `details` HTML element.
 
 Context:
-  section (griffe.docstrings.dataclasses.DocstringSectionAdmonition): The section to render.
+  section (griffe.DocstringSectionAdmonition): The section to render.
 -#}
 
 {% block logs scoped %}

src/mkdocstrings_handlers/python/templates/material/_base/docstring/attributes.html.jinja

@@ -4,7 +4,7 @@ This template renders a list of documented attributes in the format
 specified with the [`docstring_section_style`][] configuration option.
 
 Context:
-  section (griffe.docstrings.dataclasses.DocstringSectionAttributes): The section to render.
+  section (griffe.DocstringSectionAttributes): The section to render.
 -#}
 
 {% block logs scoped %}