Closed asarhaddon closed 2 years ago
Hello. I should have described the issue I was fixing more in detail. Sphinx 3.2.1 accepts both a class or an instance, but warns that version 4 will only accept classes. I suggest to postpone this merge request until your migration to sphinx 3.
The same change has been applied meanwhile in commit e7d4c550992779faef9a060d8ced6749a629ea72.
Hello @asarhaddon - we're unable to merge this: our internal infrastructure uses sphinx 1.3, which chokes on this:
(We're planning an upgrade to Sphinx 3, not 4... we're quite conservative).
Are you able to make this change in a backwards-compatible way, by any chance?