-
Notifications
You must be signed in to change notification settings - Fork 53
A mechanism for propagating docstring updates? #180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Good question. @asmeurer is updating the test suite regularly and parses the API docs to ensure docs and tests stay in sync, so I assume that updating to the latest version of the test suite should catch most/all of the changes. Maybe it'd be a matter of running one CI job against |
What info are you including in the docstrings? My script in the test suite parses the signatures and some additional metadata, but doesn't parse everything. I've argued in the past that it would be helpful to have the spec in a machine readable format #49. |
Sounds like a good idea to start with.
The whole docstring. 1-to-1 copy, with library-specific stuff appended at the end (possibly starting with
It'd be cool to have, though I am not sure if it'd increase our maintenance cost (for this repo) or not. |
This is kind of problematic with docstring parsing, e.g. numpydoc chokes on duplicate sections. Also not easy to insert examples that way, you have to create some generic one and then replace the library name. I think it'd be good to make copying content as easy as possible though. We've accumulated some technical debt here; the API docs really should have been in reST. |
Should this issue remain open? The API docs are now in |
I'll close this - having it available as regular docstrings makes them programmatically accessible, that's about as good as we can or need to do here. Thanks all! |
It's midnight so I might be asking silly or trivial questions...Let me know if I am! 😄
I think as we move forward and finalize the standard while downstream libraries start to implement the support, an apparent burden is to add docstings to all Array APIs. We certainly do not want to manually copy and paste all the docstrings from the standard, but rather we would rely on some kind of tools (should be a simple script?) to parse it.
However, is there a better way already conceived on how to propagate docstring changes to downstream? For example, if we fix a few docstrings in this repo, is there a mechanism (hopefully automated) to propagate it to all stakeholder projects (and perhaps beyond) at once? Or does it require each project's maintainer to inspect and determine from the fix if an API behavior is changed and so requires some code change as well?
The text was updated successfully, but these errors were encountered: