Skip to content

Reorder API docs: separate file per module #12727

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

Merged
merged 1 commit into from
Nov 5, 2018

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Nov 3, 2018

PR Summary

This is a start for reorganizing the API docs.

The current API docs are a mixture of plain API and additional description (e.g. https://matplotlib.org/devdocs/api/animation_api.html). The goal is to ultimately separate these.

Scope of this PR

In a first step, this PR unifies the structure of the *_api.rst files so that each module has its own file. In particular:

  • Have one file [mymodule]_api.rst for each matplotlib module.
  • The section titles are unified to be matplotlib.mymodule.
  • The modules are listed in alphabetical order in the modules section in https://matplotlib.org/devdocs/api/index.html (link points to the current docs - will be updated when CI docs ar built).

Not in this PR

This PR does not modify the way individual modules are documented; i.e. the contents of [mymodule]_api.rst remains as is except from changes of section titles and separating modules. This will be handled later on in a separate PR.

There may be changes in how we generate the API docs in the future (#11727). Either way, a clear structure is a helpful step to clean up the API docs. In that sense, this PR should be seen as an incremental improvement.

@tacaswell tacaswell added this to the v3.1 milestone Nov 4, 2018
@tacaswell
Copy link
Member

I would rather we not backport the documenation overhaul to the bug-fix branches, but open to being convinced otherwise.

I am 👍 on this.

@timhoffm
Copy link
Member Author

timhoffm commented Nov 4, 2018

I don't care too much about backporting here. I expect there's just 3.0.2 before 3.1.0 in a couple of months, so it doesn't make a big difference. OTOH, as long as the changes are limited to *._api.rst files, as in this case, the backport is trivial.

@jklymak
Copy link
Member

jklymak commented Nov 5, 2018

@jklymak jklymak merged commit 9332a06 into matplotlib:master Nov 5, 2018
@timhoffm timhoffm deleted the reorder-doc-api branch November 5, 2018 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants