Skip to content
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

Add auto-generated API docs #797

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smheidrich
Copy link
Contributor

@smheidrich smheidrich commented Sep 20, 2024

Description

Add an auto-generated API reference to Rope's docs.

Fixes #796

Details (might change as the PR evolves)

  • Currently uses sphinx-autoapi to generate the docs.
  • Lots of warnings during the docs build due to references that can't be found (mostly due to Markdown-like single-backtick inline code in docstrings), but result looks all right.
  • Would obviously benefit greatly from enabling pull request reviews in the ReadTheDocs setting.

docs/library.rst Outdated
Comment on lines 525 to 528
Have a look at ``rope.refactor`` package and its sub-modules. For
example for performing a move refactoring you can create a ``Move``
object like this:
Have a look at the `rope.refactor` package and its sub-modules. For
example for performing a move refactoring you can create an object
representing this operation (which will be an instance of e.g.
`MoveMethod`, `MoveModule`, ...) using `create_move`.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've made a few adjustments like this one in library.rst just to show the benefit of the API docs for readers of the existing docs.

Ideally, this should be done throughout the existing docs, but I'll only do that if I get the green light for the basic idea behind this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically generated API docs would be nice
1 participant