Repo2docker: RTD webhook setup

Created on 21 Feb 2019  路  6Comments  路  Source: jupyterhub/repo2docker

Could someone check the webhook integration on this repository for read the docs? See https://github.com/jupyterhub/jupyterhub/issues/2427

The last build of the r2d docs was on 16th January 2019, which seems a long time ago though I don't know how to determine when the last build should have been (is RTD smart enough to know when nothing has changed in the docs when we merge a PR?).

bug

All 6 comments

I think the webhook isn't working properly as the last build of master was a month ago but we merged changes to the docs yesterday in #583.

This needs @willingc or @minrk (I think) as you are the ones with enough privilege to look at the webhook configuration of this repository.

Added webhook.

How to update

  • Follow RTD webhook directions
  • Any maintainer listed on RTD should be able to go into Admin-> Integrations and get the webhook URL
  • You will need privileges on the GitHub repo, go into Settings -> Webhooks and add the webhook
  • Check on RTD that the webhook gave a 200 on its test ping. (Found on the RTD Admin-> Integrations -> webhook)
  • Relaunch a Build manually on RTD. Builds -> Latest -> Build Version.
  • Verify build. Note: this step should be done after every doc PR merge.

This will need to be done for every repo that has not yet been updated with a webhook in GitHub. This is necessary since Services were deprecated in GitHub.

Abusing this issue a bit as a todo list. Below the repos I've checked or tried to check. For those that I can't check I've left my name as a sign that I tried but can't (and won't check). If you are reading this please see if you can check :)

@betatim since all of your boxes are checked, can we close this one?

Thanks everyone who helped out with this tedious task!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

minrk picture minrk  路  3Comments

Morisset picture Morisset  路  5Comments

yuvipanda picture yuvipanda  路  5Comments

bitnik picture bitnik  路  3Comments

consideRatio picture consideRatio  路  6Comments