now impl-detail -> admonition directive

Author: skirpichev

pep_sphinx_extensions/__init__.py

@@ -7,7 +7,6 @@
 from typing import TYPE_CHECKING, Any
 
 from docutils.writers.html5_polyglot import HTMLTranslator
-from docutils import nodes
 from sphinx import environment
 
 from pep_sphinx_extensions.generate_rss import (
@@ -27,9 +26,6 @@
 from pep_sphinx_extensions.pep_processor.transforms import pep_references
 from pep_sphinx_extensions.pep_zero_generator.pep_index_generator import create_pep_zero
 
-from sphinx.locale import _ as sphinx_gettext
-from sphinx.util.docutils import SphinxDirective
-
 if TYPE_CHECKING:
     from sphinx.application import Sphinx
 
@@ -73,29 +69,6 @@ def set_description(
         context["description"] = "Python Enhancement Proposals (PEPs)"
 
 
-class ImplementationDetail(SphinxDirective):
-
-    has_content = True
-    final_argument_whitespace = True
-
-    # This text is copied to templates/dummy.html
-    label_text = sphinx_gettext('CPython implementation detail:')
-
-    def run(self):
-        self.assert_has_content()
-        pnode = nodes.compound(classes=['impl-detail'])
-        content = self.content
-        add_text = nodes.strong(self.label_text, self.label_text)
-        self.state.nested_parse(content, self.content_offset, pnode)
-        content = nodes.inline(pnode[0].rawsource, translatable=True)
-        content.source = pnode[0].source
-        content.line = pnode[0].line
-        content += pnode[0].children
-        pnode[0].replace_self(nodes.paragraph(
-            '', '', add_text, nodes.Text(' '), content, translatable=False))
-        return [pnode]
-
-
 def setup(app: Sphinx) -> dict[str, bool]:
     """Initialize Sphinx extension."""
 
@@ -128,8 +101,6 @@ def setup(app: Sphinx) -> dict[str, bool]:
     app.add_directive("superseded", pep_banner_directive.SupersededBanner)
     app.add_directive("withdrawn", pep_banner_directive.WithdrawnBanner)
 
-    app.add_directive('impl-detail', ImplementationDetail)
-
     # Register event callbacks
     app.connect("builder-inited", _update_config_for_builder)  # Update configuration values for builder used
     app.connect("env-before-read-docs", create_pep_zero)  # PEP 0 hook

peps/pep-0757.rst

@@ -154,7 +154,7 @@ If :c:member:`PyLongExport.digits` is not ``NULL``, a private field of the
    :c:func:`PyLong_FreeExport` must be called when the export is no longer
    needed.
 
-   .. impl-detail::
+   .. admonition:: CPython implementation detail
       This function always succeeds if *obj* is a Python :class:`int` object
       or a subclass.
 
@@ -167,7 +167,7 @@ a thin wrapper to expose Python :class:`int` internal digits array.
 
    Release the export *export_long* created by :c:func:`PyLong_Export`.
 
-   .. impl-detail::
+   .. admonition:: CPython implementation detail
       Calling :c:func:`PyLong_FreeExport` is optional if *export_long->digits*
       is ``NULL``.