[PATCH] D125744: github: Add a helpful message for issues without milestones

[Tom Stellard via Phabricator via llvm-commits]

tstellar marked 2 inline comments as done.
tstellar added inline comments.


================
Comment at: llvm/utils/git/github-automation.py:153-155
+    def issue_notify_no_milestone(self, comment) -> None:
+        message = "<!--IGNORE-->\n>{}\n\nError: Command failed due to missing milestone.".format(comment)
+        self.issue.create_comment(message)
----------------
kwk wrote:
> `<!--IGNORE-->` seems like a special comment that gets parsed here: https://github.com/llvm/llvm-project/blob/7e5f75aa37801b9e4e9f0e3fb9f0358cbafb6f63/.github/workflows/issue-release-workflow.yml#L35. I suggest to create a dedicated function for this to make this more obvious where this gets ignored. Also I think the `>` before the `{}` placeholder was not intended, right?
> `<!--IGNORE-->` seems like a special comment that gets parsed here: https://github.com/llvm/llvm-project/blob/7e5f75aa37801b9e4e9f0e3fb9f0358cbafb6f63/.github/workflows/issue-release-workflow.yml#L35. I suggest to create a dedicated function for this to make this more obvious where this gets ignored. Also I think the `>` before the `{}` placeholder was not intended, right?

The '>' character is to quote the original comment.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125744/new/

https://reviews.llvm.org/D125744