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.

Sign up for GitHub

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

Jump to bottom

Ensure graphviz is installed for readthedocs #150

Merged

mhvk merged 1 commit into liberfa:main from mhvk:readthedocs-dependency

Jun 3, 2024

Contributor

mhvk commented Jun 2, 2024

Currently, our docs are missing the inheritance diagrams - see https://pyerfa.readthedocs.io/en/latest/api.html#class-inheritance-diagram

I had similar problems in my own packages and found that what was needed was to install graphviz, so that is added here.


          Ensure graphviz is installed for readthedocs

bf4542f

avalentino approved these changes

View reviewed changes

mhvk merged commit d4d4fd5 into liberfa:main

24 checks passed

mhvk deleted the readthedocs-dependency branch

June 3, 2024 12:37

Contributor Author

mhvk commented Jun 3, 2024

Thanks, the latest build shows it worked... https://pyerfa.readthedocs.io/en/latest/api.html#class-inheritance-diagram
(not that the inheritance diagram is very exciting...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet