Check if branch is a tag before trying to update

Author: Jan200101

redbot/cogs/downloader/repo_manager.py

@@ -126,6 +126,7 @@ class Repo(RepoJSONMixin):
         "git -C {path} log --diff-filter=D --pretty=format:%H -n 1 {descendant_rev}"
         " -- {module_name}/__init__.py"
     )
+    GIT_LIST_TAGS = "git -C {path} tag -l"
 
     PIP_INSTALL = "{python} -m pip install -U -t {target_dir} {reqs}"
 
@@ -247,6 +248,28 @@ async def is_on_branch(self) -> bool:
         """
         return await self.latest_commit() == self.commit
 
+    async def is_branch_tag(self) -> bool:
+        """
+        Check if the branch assosiated with the repo is a tag.
+
+        Returns
+        -------
+        bool
+            `True` if branch is a tag or `False` otherwise
+        """
+        git_command = ProcessFormatter().format(self.GIT_LIST_TAGS, path=self.folder_path)
+        p = await self._run(git_command)
+
+        if p.returncode != 0:
+            raise errors.GitException(
+                "Git failed to get list of tags",
+                git_command,
+            )
+
+        tags = p.stdout.decode(**DECODE_PARAMS).strip().split("\n")
+
+        return self.branch in tags
+
     async def _get_file_update_statuses(
         self, old_rev: str, new_rev: Optional[str] = None
     ) -> Dict[str, str]:
@@ -815,6 +838,9 @@ async def update(self) -> Tuple[str, str]:
         -------
         `UpdateError` - if git pull results with non-zero exit code
         """
+        if await self.is_branch_tag():
+            return self.commit, self.commit
+
         old_commit = await self.latest_commit()
 
         await self.hard_reset()

tests/cogs/downloader/test_downloader.py

@@ -324,6 +324,7 @@ async def test_update(mocker, repo):
     new_commit = "a0ccc2390883c85a361f5a90c72e1b07958939fa"
     m = _mock_run(mocker, repo, 0)
     _mock_setup_repo(mocker, repo, new_commit)
+    mocker.patch.object(repo, "is_branch_tag", autospec=True, return_value=False)
     mocker.patch.object(repo, "latest_commit", autospec=True, return_value=old_commit)
     mocker.patch.object(repo, "hard_reset", autospec=True, return_value=None)
     ret = await repo.update()