Fix incomplete cleanup in autodoc_enhancements.py - #224

Please can someone approve the workflows?

Codecov Report

❌ Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.62%. Comparing base (1b47aca) to head (e15207f).
⚠️ Report is 31 commits behind head on main.

@@            Coverage Diff             @@
##             main     #224      +/-   ##
==========================================
- Coverage   89.69%   89.62%   -0.08%     
==========================================
  Files          31       31              
  Lines        1262     1253       -9     
==========================================
- Hits         1132     1123       -9     
  Misses        130      130              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I think this is hitting one of the errors I saw in #223 that I do not understand.

      File "sphinx/events.py", line 402, in emit
        raise ExtensionError(
        ...<4 lines>...
        ) from exc
    sphinx.errors.ExtensionError: Handler <function process_automodsumm_generation at 0x7f73f1415580>
        for event 'builder-inited' threw an exception (exception: 'class')

@pllim slightly hubristicly, I put up this PR without running tests, as I didn't want to set up a full environment for the project!

Using Sphinx's --pdb flag, we can see that the second problem is the following:

      File "...\sphinx-automodapi\sphinx_automodapi\utils.py", line 256, in get_documenter
        retval = app.registry.documenters[obj_type]
                 ~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^
    KeyError: 'class'

I've thus applied the suggestion I made elsewhere to change the vendored get_documenter() function to get_object_type(), which works for me locally.

A

xref astropy/sphinx-astropy#86

Thank you so much @AA-Turner!

Using Sphinx's --pdb flag, we can see that the second problem is the following:
And @pllim - I think we should add this to tox command; less cryptic errors would be such and improvement here :)

I'm not sure it would help, it's for interactive debugging only: https://www.sphinx-doc.org/en/master/man/sphinx-build.html#cmdoption-sphinx-build-P

I'm not sure it would help, it's for interactive debugging only: sphinx-doc.org/en/master/man/sphinx-build.html#cmdoption-sphinx-build-P

Ahh, never mind it then.