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

[docs] Sphinx API hosted on GitHub Pages #832

Merged
merged 11 commits into from
Aug 11, 2020

Conversation

j0yu
Copy link
Contributor

@j0yu j0yu commented Jan 8, 2020

Sphinx

Build Sphinx API docs and host it using GitHub Pages for the current repository:

https://wwfxuk.github.io/rez/

  • Switched to use ReadTheDocs theme
  • Publish only on master branch
    • Built HTML available as CI artefacts
  • Platform independent way to build docs python docs/entrypoint.py (add --no-docker to just install locally)

Thought this might be a starting point to have some API docs available:

  • When developing rez plugins, it will be more intuitive to have the references available
  • Help keep in check the docstrings written can actually be compiled by Sphinx

@nerdvegas nerdvegas merged commit 985cdc3 into AcademySoftwareFoundation:master Aug 11, 2020
@j0yu j0yu deleted the build-sphinx branch August 11, 2020 11:09
@j0yu
Copy link
Contributor Author

j0yu commented Aug 11, 2020

About 60d21f4, I originally included it just to make sure users on any platform/python version can build the docs for themselves locally.

It's to make sure we don't end up with any unintended surprises in any API docs Python scripts if we start adding any in the future.

That said, if anyone PRs anything, I guess as long as it builds on GitHub and releases, that is what really matters at the end of the day. People can download the generated artefact and view the static, generated pages locally if they want to check.

@j0yu
Copy link
Contributor Author

j0yu commented Aug 11, 2020

Also, I find the GitHub Pages settings can be a bit stubborn and either:

  • Take a while to take effect or,
  • Needs a couple of cycles of setting/un-setting it for it to take effect
    image

@nerdvegas
Copy link
Contributor

nerdvegas commented Aug 11, 2020 via email

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.

2 participants