Update some comments after noticing how Build.identifier/.verbose_name fields are used

Author: benjaoming

readthedocs/api/v2/views/integrations.py

@@ -15,11 +15,7 @@
 from rest_framework.status import HTTP_400_BAD_REQUEST
 from rest_framework.views import APIView
 
-from readthedocs.builds.constants import (
-    EXTERNAL_VERSION_STATE_CLOSED,
-    EXTERNAL_VERSION_STATE_OPEN,
-    LATEST,
-)
+from readthedocs.builds.constants import LATEST
 from readthedocs.core.signals import webhook_bitbucket, webhook_github, webhook_gitlab
 from readthedocs.core.views.hooks import (
     build_branches,
@@ -190,7 +186,7 @@ def get_response_push(self, project, branches):
 
         :param project: Project instance
         :type project: Project
-        :param branches: List of branch names to build
+        :param branches: List of branch/tag names to build
         :type branches: list(str)
         """
         to_build, not_building = build_branches(project, branches)

readthedocs/builds/tasks.py

@@ -292,8 +292,10 @@ def sync_versions_task(project_pk, tags_data, branches_data, **kwargs):
     Creates new Version objects for tags/branches that aren't tracked in the database,
     and deletes Version objects for tags/branches that don't exists in the repository.
 
-    :param tags_data: List of dictionaries with ``verbose_name`` and ``identifier``.
-    :param branches_data: Same as ``tags_data`` but for branches.
+    :param tags_data: List of dictionaries with ``verbose_name`` and ``identifier``
+                      Example: ("v1.0.0", "67a9035990f44cb33091026d7453d51606350519").
+    :param branches_data: Same as ``tags_data`` but for branches (branch name, branch identifier).
+                          Example: ("latest", "main")
     :returns: `True` or `False` if the task succeeded.
     """
     project = Project.objects.get(pk=project_pk)