Fix checking of PR status (#10085)
* Fix checking of PR status
This was previously checking the status of the build object,
but I think we want to check the status that is passed in,
and only link to the built docs when the build was successful.
* use positional arguments, doesn't seem like celery can use them as kwargs
* Revert remaining kwargs calls to positional args
---------
Co-authored-by: Benjamin Bach <benjamin@readthedocs.org>
