Put Sphinx build in the CI

[pavelkomarov]

Just looked at the .ymls, and there's no sphinx build happening there. So the existing docs must have been built offline and pushed up manually. It would be better to have them built automatically and deployed to the gh-pages branch, both to get them out of the main branch and so that they reflect any changes to the code automatically and immediately.

Maybe something like this is indeed happening. I haven't used readthedocs.io before.

[pavelkomarov]

It appears readthedocs takes the repo and builds and deploys everything on its end. But it is possible to integrate with CI so this process is triggered by code changes and doesn't need to be done manually.

https://docs.readthedocs.com/platform/latest/tutorial/index.html

[pavelkomarov]

I think readthedocs does indeed monitor the main branch and rebuild automatically when changes are made. If I can confirm this is the case for this repo, we can close this one.

[pavelkomarov]

@luckystarufo, we merged #77, but readthedocs didn't rebuild. Your avatar appears at https://app.readthedocs.org/projects/pynumdiff/builds/, so my guess is you set it up. Do you have permissions to make this fully integrated and automatic? Or permissions to add my newly-created readthedocs profile to the project so I can try to set this up?

[pavelkomarov]

https://docs.readthedocs.com/platform/stable/reference/git-integration.html#integration-creation

I think to set it up myself I would need admin rights on this repo.

[pavelkomarov]

Yeah, I can't actually take care of this. @luckystarufo, we need you. I've opened #86 in an attempt to help with automatic integration.

[pavelkomarov]

Looks like the readthedocs is actually connected to Yuying's fork. We should probably connect it to the original repo instead.

[pavelkomarov]

Update: Yuying is happy to hand us the reigns, but he won't be able to do so until 7/1, so we'll wait. No rush.

[pavelkomarov]

Okay, at last today we connected the canonical repo to readthedocs. It took some futzing around with the configuration file to get things to build correctly, and I had to refresh the webhook to get it to kick off automatically on changes to master, probably because the old webhook was associated with Yuying's fork, but it's working now.