Merge pull request #6877 from readthedocs/configurable-status-name

Make the status name in CI configurable via setting

Author: ericholscher

readthedocs/builds/constants.py

@@ -100,8 +100,6 @@
     },
 }
 
-RTD_BUILD_STATUS_API_NAME = 'continuous-documentation/read-the-docs'
-
 GITHUB_EXTERNAL_VERSION_NAME = 'Pull Request'
 GITLAB_EXTERNAL_VERSION_NAME = 'Merge Request'
 GENERIC_EXTERNAL_VERSION_NAME = 'External Version'

readthedocs/oauth/services/github.py

@@ -15,7 +15,6 @@
 from readthedocs.builds.constants import (
     BUILD_STATUS_SUCCESS,
     SELECT_BUILD_STATUS,
-    RTD_BUILD_STATUS_API_NAME
 )
 from readthedocs.integrations.models import Integration
 
@@ -438,7 +437,7 @@ def send_build_status(self, build, commit, state):
             'state': github_build_state,
             'target_url': target_url,
             'description': description,
-            'context': RTD_BUILD_STATUS_API_NAME
+            'context': settings.RTD_BUILD_STATUS_API_NAME
         }
 
         resp = None

readthedocs/oauth/services/gitlab.py

@@ -11,7 +11,6 @@
 
 from readthedocs.builds.constants import (
     BUILD_STATUS_SUCCESS,
-    RTD_BUILD_STATUS_API_NAME,
     SELECT_BUILD_STATUS,
 )
 from readthedocs.builds import utils as build_utils
@@ -524,7 +523,7 @@ def send_build_status(self, build, commit, state):
             'state': gitlab_build_state,
             'target_url': target_url,
             'description': description,
-            'context': RTD_BUILD_STATUS_API_NAME
+            'context': settings.RTD_BUILD_STATUS_API_NAME
         }
         url = self.adapter.provider_base_url
 

readthedocs/settings/base.py

@@ -99,6 +99,7 @@ class CommunityBaseSettings(Settings):
     RTD_STABLE_VERBOSE_NAME = 'stable'
     RTD_CLEAN_AFTER_BUILD = False
     RTD_MAX_CONCURRENT_BUILDS = 4
+    RTD_BUILD_STATUS_API_NAME = 'docs/readthedocs'
 
     # Database and API hitting settings
     DONT_HIT_API = False