WIP

kognity · Jul 10, 2023 · 360ca32 · 360ca32
1 parent ebcf36a
commit 360ca32
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 13 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -17,9 +17,9 @@ jobs:
       # - uses: kognity/pr-update-action@master
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
-          title: My new PR Title
-          title-prefix: PFX-999
-          title-suffix: "[BOT]"
-          body: My new PR Body
+          # title: My new PR Title
+          # title-prefix: PFX-999
+          # title-suffix: "[BOT]"
+          # body: My new PR Body
           body-prefix: Some prefix on the body
           body-suffix: Some suffix on the body
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -24,7 +24,7 @@ async function run() {
     if (newPrTitle) updateRequest.title = newPrTitle;
     if (newPrBody) updateRequest.body = newPrBody;
 
-    core.info(`Request: \n${JSON.stringify(updateRequest)}.`);
+    core.info(`Request: \n${JSON.stringify(updateRequest, null, 4)}.`);
 
     if (!newPrTitle && !newPrBody) {
       core.warning('No changes made to PR title or body. Exiting.');

diff --git a/src/utils.ts b/src/utils.ts
@@ -37,15 +37,28 @@ export function updateBody(
   inputs: ReturnType<typeof getInputs>,
   prBody: string
 ): string | undefined {
-  const {body, bodyPrefix, bodySuffix, bodyConcatNewLine} = inputs;
+  const {
+    body: bodyLines,
+    bodyPrefix: bodyPrefixLines,
+    bodySuffix: bodySuffixLines,
+    bodyConcatNewLine,
+  } = inputs;
+  const body = bodyLines.join('\n').trim();
+  const bodyPrefix = bodyPrefixLines.join('\n').trim();
+  const bodySuffix = bodySuffixLines.join('\n').trim();
   const concatStrategy = bodyConcatNewLine ? '\n' : ' ';
-  core.info(`Current PR body: ${prBody}.`);
+  core.info(`Current PR body: \n${prBody}.`);
+  core.warning(
+    `This should be false, and the value is: ${JSON.stringify(
+      body || bodyPrefix || bodySuffix
+    )}.`
+  );
 
   if (body || bodyPrefix || bodySuffix) {
     const newBody = [bodyPrefix, body || prBody, bodySuffix]
       .filter(Boolean)
       .join(concatStrategy);
-    core.info(`New body: ${newBody}.`);
+    core.info(`New body: \n${newBody}`);
     core.setOutput('new-body', newBody);
     return newBody;
   }