search

cabouman / mbircone

BSD 3-Clause "New" or "Revised" License
11 stars 9 forks source link

Fix sphinx requirements #150

Closed cabouman closed 1 year ago

cabouman commented 1 year ago

I changed the file doc mbircone/docs/requirements.txt in a way that seems to fix the problems with Sphinx. I based the changes on Qiuchen's repository that seems to work on readthedocs. Please check to see if you agree.

dyang37 commented 1 year ago

This fix works for my local docs build. I'm not sure whether this would work for the online readthedocs. Is there a way to do an online readthedocs build from this branch and see if it's successful?

cabouman commented 1 year ago

It also works for Quichen's readthedocs page, so I'm going to give it a try.

bwohlberg commented 1 year ago

This fix works for my local docs build. I'm not sure whether this would work for the online readthedocs. Is there a way to do an online readthedocs build from this branch and see if it's successful?

Yes. Once can configure readthedocs to automatically build the docs as part of the CI for a PR.