Ashley Whetter
Ashley Whetter
Nice find! I'm not sure how I missed that before. But a small end-to-end test is working as it should for me now. > which is quite ugly, in the...
I've just squashed the commits back up, so this is now ready for review.
By chance, stubgen works, but only when the signature headings are included. stubgen isn't capable of parsing the backslash separated signatures at the start of the docstring, so it ignores...
It would be less intrusive for pybind, yes. However I'm trying to make everything agree on a convention that is also used outside of pybind, rather than have everything support...
For that to work I think we would have to change the signature parsing of Sphinx so that it matches what stubgen does where it tokenises that string and properly...
Good news! Sphinx v4 will support the function signatures separated without backslashes. I've updated this pull request to not include separating backslashes. I've done another small end to end test...
> Apparently the docstrings are mined to build .pyi files, in this case picking up the function signatures twice. Is stubgen being used to do this and does each overload...
As for disabling this behaviour by default, it will require adding another option and I don't love that this pull request would then increase the number of options from two...
I've made the requested changes. The test failures look unrelated to my changes. I did try out using a single new option to switch between the current docstrings, and a...
I can separate out my latest changes from the original commits if you like? I wasn't sure if you would squash them or if that was up to me.