amoeba commented on PR #2890:
URL: https://github.com/apache/arrow-adbc/pull/2890#issuecomment-3049604755

   > > What do you think about breaking the driver status info into its own 
section and linking the table header to that section?
   > 
   > Do you mean linking the table rows? I do intend to do that, just haven't 
updated all the doc pages. I'm not sure the status info is "heavy" enough to 
warrant an entire separate section though. I will update the badges in the 
Sphinx docs to have a label (so 'Status: Stable' not just 'Stable'). 
Unfortunately Bootstrap badges don't seem to support the two-tone thing that 
shields.io does
   
   Something like this:
   
   <details>
   <summary>patch</summary>
   
   ```patch
   diff --git a/docs/source/driver/status.rst b/docs/source/driver/status.rst
   index ec05572bd..06bab08ea 100644
   --- a/docs/source/driver/status.rst
   +++ b/docs/source/driver/status.rst
   @@ -28,10 +28,6 @@ Driver Implementation Status
    Implementation Status
    =====================
    
   -**Experimental** drivers are not feature-complete and the implementation is 
still progressing.
   -**Beta** drivers are (mostly) feature-complete but have only been available 
for a short time.
   -**Stable** drivers are (mostly) feature-complete (as much as possible for 
the underlying database) and have been available/tested for a while.
   -
    .. note::
    
       Drivers that support C/C++ can also be used from C#, GLib, Go, Python, R,
   @@ -59,6 +55,23 @@ Implementation Status
                 `DuckDB documentation
                 <https://duckdb.org/docs/stable/clients/adbc.html>`_ for 
details.
    
   +
   +.. _driver_status:
   +
   +Driver Status
   +=============
   +
   +The drivers in the above table vary with respect to their maturity and have 
been categorized according to the following groupings:
   +
   +**Stable**
   +    Mostly feature-complete (as much as possible for the underlying 
database) and have been available/tested for a while.
   +
   +**Beta**
   +    Mostly feature-complete but have only been available for a short time.
   +
   +**Experimental**
   +    Not feature-complete and the implementation is still progressing.
   +
    Feature Support
    ===============
    
   diff --git a/docs/source/ext/adbc_misc.py b/docs/source/ext/adbc_misc.py
   index cf6aa3aca..9a98d95ef 100644
   --- a/docs/source/ext/adbc_misc.py
   +++ b/docs/source/ext/adbc_misc.py
   @@ -298,7 +298,7 @@ class DriverStatusTableDirective(SphinxDirective):
                "",
                "   * - Vendor",
                "     - Implementation",
   -            "     - Driver Status",
   +            "     - :ref:`driver_status`",
                "     - Packages [#packages]_",
                "",
            ]
   ```
   </details>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to