From 990db610f19c3ef9f0fac5cc071d3e982f83ec8b Mon Sep 17 00:00:00 2001
From: tison <wander4096@gmail.com>
Date: Tue, 14 May 2024 16:48:16 +0800
Subject: [PATCH] fixup

Signed-off-by: tison <wander4096@gmail.com>
---
 cyborg/bin/follow-up-docs-issue.ts | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/cyborg/bin/follow-up-docs-issue.ts b/cyborg/bin/follow-up-docs-issue.ts
index b915b541a4fe..0b2e192579d5 100644
--- a/cyborg/bin/follow-up-docs-issue.ts
+++ b/cyborg/bin/follow-up-docs-issue.ts
@@ -22,6 +22,8 @@ import {PullRequestEditedEvent, PullRequestEvent, PullRequestOpenedEvent} from "
 import {RequestError} from "@octokit/request-error";
 
 const needFollowUpDocs = "[x] This PR requires documentation updates."
+const labelDocsNotRequired = "docs-not-required"
+const labelDocsRequired = "docs-required"
 
 async function main() {
     if (!context.payload.pull_request) {
@@ -41,17 +43,17 @@ async function main() {
     }
     const followUpDocs = checkPullRequestEvent(payload)
     if (followUpDocs) {
+        core.info("Follow up docs.")
         await client.rest.issues.removeLabel({
-            owner, repo, issue_number: number, name: 'docs-not-need',
+            owner, repo, issue_number: number, name: labelDocsNotRequired,
         }).catch((e: RequestError) => {
             if (e.status != 404) {
                 throw e;
-            } else {
-                core.debug("Labels to be removed do not exist.")
             }
+            core.debug(`Label ${labelDocsNotRequired} not exist.`)
         })
         await client.rest.issues.addLabels({
-            owner, repo, issue_number: number, labels: ['docs-required'],
+            owner, repo, issue_number: number, labels: [labelDocsRequired],
         })
         await docsClient.rest.issues.create({
             owner: 'GreptimeDB',
@@ -59,19 +61,21 @@ async function main() {
             title: `Update docs for ${title}`,
             body: `A document change request is generated from ${html_url}`,
             assignee: actor,
+        }).then((res) => {
+            core.info(`Created issue ${res.data}`)
         })
     } else {
+        core.info("No need to follow up docs.")
         await client.rest.issues.removeLabel({
-            owner, repo, issue_number: number, name: 'docs-required'
+            owner, repo, issue_number: number, name: labelDocsRequired
         }).catch((e: RequestError) => {
             if (e.status != 404) {
                 throw e;
-            } else {
-                core.debug("Labels to be removed do not exist.")
             }
+            core.debug(`Label ${labelDocsRequired} not exist.`)
         })
         await client.rest.issues.addLabels({
-            owner, repo, issue_number: number, labels: ['docs-not-need'],
+            owner, repo, issue_number: number, labels: [labelDocsNotRequired],
         })
     }
 }