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

DOC: Fix Sphinx documentation #180

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented Oct 17, 2023

Fix Sphinx documentation:

  • Add a pre-build task to the readthedocs config file in order to
    generate the rst files from the package docstrings so that Sphinx
    build the HTML files from them.
  • Add the napoleon extension to the Sphinx config file so that it can
    parse NumPy and Google style docstrings.
  • Fix the index.rst file so that the package API is effectively
    displayed: set whitematterpackage as the entry for the reference
    section.
  • Set the bin and utilities packages as the entries for the scripts
    section in the index.rst file.

@jhlegarreta jhlegarreta force-pushed the FixSphinxDocumentation branch from d327c4a to 407615a Compare October 22, 2023 18:13
@jhlegarreta jhlegarreta marked this pull request as ready for review October 22, 2023 18:15
@jhlegarreta jhlegarreta marked this pull request as draft October 22, 2023 18:15
@jhlegarreta jhlegarreta force-pushed the FixSphinxDocumentation branch from 407615a to 7acafa0 Compare October 24, 2023 01:09
Fix Sphinx documentation:
- Add a pre-build task to the `readthedocs` config file in order to
  generate the `rst` files from the package docstrings so that Sphinx
  build the HTML files from them.
- Add the `napoleon` extension to the Sphinx config file so that it can
  parse NumPy and Google style docstrings.
- Fix the `index.rst` file so that the package API is effectively
  displayed: set `whitematterpackage` as the entry for the reference
  section.
- Set the `bin` and `utilities` packages as the entries for the scripts
  section in the `index.rst` file.
@jhlegarreta jhlegarreta force-pushed the FixSphinxDocumentation branch from 7acafa0 to a45a7ae Compare October 24, 2023 01:10
@jhlegarreta jhlegarreta marked this pull request as ready for review October 24, 2023 01:15
@jhlegarreta
Copy link
Contributor Author

The patch set is doing its job: the package documentation is now displayed.

Further improvements are left for separate PRs.

Merging.

@jhlegarreta jhlegarreta merged commit 5b28ab2 into SlicerDMRI:master Oct 24, 2023
@jhlegarreta jhlegarreta deleted the FixSphinxDocumentation branch October 24, 2023 01:17
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.

1 participant