Merge pull request #110 from mikepenz/develop

dev -> main
2020-12-02 09:31:05 +01:00
parent f79b637452 f6de4dfc32
commit a2859f58b7
11 changed files with 11882 additions and 519 deletions
@@ -1,4 +1,4 @@
-import * as path from 'path';
+import * as path from 'path'
 import * as process from 'process'
 import * as cp from 'child_process'

@@ -11,7 +11,7 @@ test('missing values should result in failure', () => {
  }
  try {
    cp.execSync(`node ${ip}`, options).toString()
-    fail("Should not succeed, because values miss")
+    fail('Should not succeed, because values miss')
  } catch (error) {
    console.log(`correctly failed due to: ${error}`)
  }
@@ -24,11 +24,11 @@ test('missing values should result in failure', () => {
  process.env['INPUT_REPO'] = 'release-changelog-builder-action'
  process.env['INPUT_FROMTAG'] = 'v0.3.0'
  process.env['INPUT_TOTAG'] = 'v0.5.0'
-  
+
  const ip = path.join(__dirname, '..', 'lib', 'main.js')
  const options: cp.ExecSyncOptions = {
    env: process.env
  }
  const result = cp.execSync(`node ${ip}`, options).toString()
  // should succeed
-})
+})
@@ -1,16 +1,15 @@
-import { ReleaseNotes } from '../src/releaseNotes'
-import { resolveConfiguration } from '../src/utils';
-import { Octokit } from '@octokit/rest';
+import {ReleaseNotes} from '../src/releaseNotes'
+import {resolveConfiguration} from '../src/utils'
+import {Octokit} from '@octokit/rest'
+
+jest.setTimeout(180000)

 // load octokit instance
 const octokit = new Octokit({
  auth: `token ${process.env.GITHUB_TOKEN}`
 })

-
 it('Should have empty changelog (tags)', async () => {
-  jest.setTimeout(180000)
-
  const configuration = resolveConfiguration('', 'configs/configuration.json')
  const releaseNotes = new ReleaseNotes(octokit, {
    owner: 'mikepenz',
@@ -27,8 +26,6 @@ it('Should have empty changelog (tags)', async () => {
 })

 it('Should match generated changelog (tags)', async () => {
-  jest.setTimeout(180000)
-
  const configuration = resolveConfiguration('', 'configs/configuration.json')
  const releaseNotes = new ReleaseNotes(octokit, {
    owner: 'mikepenz',
@@ -50,9 +47,10 @@ it('Should match generated changelog (tags)', async () => {
 })

 it('Should match generated changelog (refs)', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_all_placeholders.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_all_placeholders.json'
+  )
  const releaseNotes = new ReleaseNotes(octokit, {
    owner: 'mikepenz',
    repo: 'release-changelog-builder-action',
@@ -81,9 +79,10 @@ nhoelzl
 })

 it('Should match ordered ASC', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_asc.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_asc.json'
+  )
  const releaseNotes = new ReleaseNotes(octokit, {
    owner: 'mikepenz',
    repo: 'release-changelog-builder-action',
@@ -95,13 +94,16 @@ it('Should match ordered ASC', async () => {

  const changeLog = await releaseNotes.pull()
  console.log(changeLog)
-  expect(changeLog).toStrictEqual(`## 🚀 Features\n\n22\n24\n25\n26\n28\n\n## 🐛 Fixes\n\n23\n\n`)
+  expect(changeLog).toStrictEqual(
+    `## 🚀 Features\n\n22\n24\n25\n26\n28\n\n## 🐛 Fixes\n\n23\n\n`
+  )
 })

 it('Should match ordered DESC', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_desc.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_desc.json'
+  )
  const releaseNotes = new ReleaseNotes(octokit, {
    owner: 'mikepenz',
    repo: 'release-changelog-builder-action',
@@ -113,5 +115,7 @@ it('Should match ordered DESC', async () => {

  const changeLog = await releaseNotes.pull()
  console.log(changeLog)
-  expect(changeLog).toStrictEqual(`## 🚀 Features\n\n28\n26\n25\n24\n22\n\n## 🐛 Fixes\n\n23\n\n`)
-})
+  expect(changeLog).toStrictEqual(
+    `## 🚀 Features\n\n28\n26\n25\n24\n22\n\n## 🐛 Fixes\n\n23\n\n`
+  )
+})
@@ -1,9 +1,9 @@
-import { resolveConfiguration } from '../src/utils';
-import { ReleaseNotesBuilder } from '../src/releaseNotesBuilder';
+import {resolveConfiguration} from '../src/utils'
+import {ReleaseNotesBuilder} from '../src/releaseNotesBuilder'
+
+jest.setTimeout(180000)

 it('Should match generated changelog (unspecified fromTag)', async () => {
-  jest.setTimeout(180000)
-
  const configuration = resolveConfiguration('', 'configs/configuration.json')
  const releaseNotesBuilder = new ReleaseNotesBuilder(
    null,
@@ -28,8 +28,6 @@ it('Should match generated changelog (unspecified fromTag)', async () => {
 })

 it('Should use empty placeholder', async () => {
-  jest.setTimeout(180000)
-
  const configuration = resolveConfiguration('', 'configs/configuration.json')
  const releaseNotesBuilder = new ReleaseNotesBuilder(
    null,
@@ -49,9 +47,10 @@ it('Should use empty placeholder', async () => {
 })

 it('Should fill empty placeholders', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_empty_all_placeholders.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_empty_all_placeholders.json'
+  )
  const releaseNotesBuilder = new ReleaseNotesBuilder(
    null,
    '.',
@@ -66,13 +65,16 @@ it('Should fill empty placeholders', async () => {

  const changeLog = await releaseNotesBuilder.build()
  console.log(changeLog)
-  expect(changeLog).toStrictEqual(`mikepenz\nrelease-changelog-builder-action\nv0.0.2\nv0.0.3`)
+  expect(changeLog).toStrictEqual(
+    `mikepenz\nrelease-changelog-builder-action\nv0.0.2\nv0.0.3`
+  )
 })

 it('Should fill `template` placeholders', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_empty_all_placeholders.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_empty_all_placeholders.json'
+  )
  const releaseNotesBuilder = new ReleaseNotesBuilder(
    null,
    '.',
@@ -87,13 +89,16 @@ it('Should fill `template` placeholders', async () => {

  const changeLog = await releaseNotesBuilder.build()
  console.log(changeLog)
-  expect(changeLog).toStrictEqual(`## 🧪 Tests\n\n- [CI] Specify Test Case\n   - PR: #10\n\n\n\n\nmikepenz\nrelease-changelog-builder-action\nv0.0.1\nv0.0.3\n1\n0\n0`)
+  expect(changeLog).toStrictEqual(
+    `## 🧪 Tests\n\n- [CI] Specify Test Case\n   - PR: #10\n\n\n\n\nmikepenz\nrelease-changelog-builder-action\nv0.0.1\nv0.0.3\n1\n0\n0`
+  )
 })

 it('Should fill `template` placeholders, ignore', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_empty_all_placeholders.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_empty_all_placeholders.json'
+  )
  const releaseNotesBuilder = new ReleaseNotesBuilder(
    null,
    '.',
@@ -108,14 +113,16 @@ it('Should fill `template` placeholders, ignore', async () => {

  const changeLog = await releaseNotesBuilder.build()
  console.log(changeLog)
-  expect(changeLog).toStrictEqual(`## 🚀 Features\n\n- Enhance sorting by using proper semver\n   - PR: #51\n\n## 🧪 Tests\n\n- Improve test cases\n   - PR: #49\n\n\n- Bump @types/node from 14.11.8 to 14.11.10\n   - PR: #47\n- Adjust code to move fromTag resolving to main.ts\n   - PR: #48\n- dev -> main\n   - PR: #52\n- Update package.json to updated description\n   - PR: #53\n- dev -> main\n   - PR: #54\n\n- New additional placeholders for \`template\` and \`empty_template\`\n   - PR: #50\n\nmikepenz\nrelease-changelog-builder-action\nv0.9.1\nv0.9.5\n2\n5\n1`)
+  expect(changeLog).toStrictEqual(
+    `## 🚀 Features\n\n- Enhance sorting by using proper semver\n   - PR: #51\n\n## 🧪 Tests\n\n- Improve test cases\n   - PR: #49\n\n\n- Bump @types/node from 14.11.8 to 14.11.10\n   - PR: #47\n- Adjust code to move fromTag resolving to main.ts\n   - PR: #48\n- dev -> main\n   - PR: #52\n- Update package.json to updated description\n   - PR: #53\n- dev -> main\n   - PR: #54\n\n- New additional placeholders for \`template\` and \`empty_template\`\n   - PR: #50\n\nmikepenz\nrelease-changelog-builder-action\nv0.9.1\nv0.9.5\n2\n5\n1`
+  )
 })

-
 it('Uncategorized category', async () => {
-  jest.setTimeout(180000)
-
-  const configuration = resolveConfiguration('', 'configs_test/configuration_uncategorized_category.json')
+  const configuration = resolveConfiguration(
+    '',
+    'configs_test/configuration_uncategorized_category.json'
+  )
  const releaseNotesBuilder = new ReleaseNotesBuilder(
    null,
    '.',
@@ -130,5 +137,7 @@ it('Uncategorized category', async () => {

  const changeLog = await releaseNotesBuilder.build()
  console.log(changeLog)
-  expect(changeLog).toStrictEqual(`## 🚀 Features\n\n- Enhance sorting by using proper semver\n   - PR: #51\n\n## 📦 Uncategorized\n\n- Bump @types/node from 14.11.8 to 14.11.10\n   - PR: #47\n- Adjust code to move fromTag resolving to main.ts\n   - PR: #48\n- Improve test cases\n   - PR: #49\n- dev -> main\n   - PR: #52\n- Update package.json to updated description\n   - PR: #53\n- dev -> main\n   - PR: #54\n\n\n\nUncategorized:\n- Bump @types/node from 14.11.8 to 14.11.10\n   - PR: #47\n- Adjust code to move fromTag resolving to main.ts\n   - PR: #48\n- Improve test cases\n   - PR: #49\n- dev -> main\n   - PR: #52\n- Update package.json to updated description\n   - PR: #53\n- dev -> main\n   - PR: #54\n\n\nIgnored:\n- New additional placeholders for \`template\` and \`empty_template\`\n   - PR: #50\n\n\n6\n1`)
-})
+  expect(changeLog).toStrictEqual(
+    `## 🚀 Features\n\n- Enhance sorting by using proper semver\n   - PR: #51\n\n## 📦 Uncategorized\n\n- Bump @types/node from 14.11.8 to 14.11.10\n   - PR: #47\n- Adjust code to move fromTag resolving to main.ts\n   - PR: #48\n- Improve test cases\n   - PR: #49\n- dev -> main\n   - PR: #52\n- Update package.json to updated description\n   - PR: #53\n- dev -> main\n   - PR: #54\n\n\n\nUncategorized:\n- Bump @types/node from 14.11.8 to 14.11.10\n   - PR: #47\n- Adjust code to move fromTag resolving to main.ts\n   - PR: #48\n- Improve test cases\n   - PR: #49\n- dev -> main\n   - PR: #52\n- Update package.json to updated description\n   - PR: #53\n- dev -> main\n   - PR: #54\n\n\nIgnored:\n- New additional placeholders for \`template\` and \`empty_template\`\n   - PR: #50\n\n\n6\n1`
+  )
+})
@@ -1,87 +1,93 @@
-import { resolveConfiguration } from '../src/utils';
-import { ReleaseNotesBuilder } from '../src/releaseNotesBuilder';
-import { TagInfo, sortTags } from '../src/tags';
+import {resolveConfiguration} from '../src/utils'
+import {ReleaseNotesBuilder} from '../src/releaseNotesBuilder'
+import {TagInfo, sortTags} from '../src/tags'
+
+jest.setTimeout(180000)

 it('Should order tags correctly using semver', async () => {
-  jest.setTimeout(180000)
-
  const tags: TagInfo[] = [
-    { name: "2020.4.0", commit: "" },
-    { name: "2020.4.0-rc02", commit: "" },
-    { name: "2020.3.2", commit: "" },
-    { name: "v2020.3.1", commit: "" },
-    { name: "2020.3.1-rc03", commit: "" },
-    { name: "2020.3.1-rc01", commit: "" },
-    { name: "2020.3.1-b01", commit: "" },
-    { name: "v2020.3.0", commit: "" }
+    {name: '2020.4.0', commit: ''},
+    {name: '2020.4.0-rc02', commit: ''},
+    {name: '2020.3.2', commit: ''},
+    {name: 'v2020.3.1', commit: ''},
+    {name: '2020.3.1-rc03', commit: ''},
+    {name: '2020.3.1-rc01', commit: ''},
+    {name: '2020.3.1-b01', commit: ''},
+    {name: 'v2020.3.0', commit: ''}
  ]

  const tagResolver = {
-    method: "semver"
+    method: 'semver'
  }
-  const sorted = sortTags(tags, tagResolver).map(function (tag) {
-    return tag.name
-  }).join(",")
+  const sorted = sortTags(tags, tagResolver)
+    .map(function (tag) {
+      return tag.name
+    })
+    .join(',')

-  expect(sorted).toStrictEqual(`2020.4.0,2020.4.0-rc02,2020.3.2,v2020.3.1,2020.3.1-rc03,2020.3.1-rc01,2020.3.1-b01,v2020.3.0`)
+  expect(sorted).toStrictEqual(
+    `2020.4.0,2020.4.0-rc02,2020.3.2,v2020.3.1,2020.3.1-rc03,2020.3.1-rc01,2020.3.1-b01,v2020.3.0`
+  )
 })

-
 it('Should order tags correctly using semver', async () => {
-  jest.setTimeout(180000)
-
  const tags: TagInfo[] = [
-    { name: "0.0.1", commit: "" },
-    { name: "0.0.1-rc01", commit: "" },
-    { name: "0.1.0", commit: "" },
-    { name: "0.1.0-b01", commit: "" },
-    { name: "1.0.0", commit: "" },
-    { name: "1.0.0-a01", commit: "" },
-    { name: "2.0.0", commit: "" },
-    { name: "10.0.0", commit: "" },
-    { name: "10.1.0", commit: "" },
-    { name: "10.1.0-2", commit: "" },
-    { name: "20.0.2", commit: "" },
-    { name: "100.0.0", commit: "" },
-    { name: "1000.0.0", commit: "" },
+    {name: '0.0.1', commit: ''},
+    {name: '0.0.1-rc01', commit: ''},
+    {name: '0.1.0', commit: ''},
+    {name: '0.1.0-b01', commit: ''},
+    {name: '1.0.0', commit: ''},
+    {name: '1.0.0-a01', commit: ''},
+    {name: '2.0.0', commit: ''},
+    {name: '10.0.0', commit: ''},
+    {name: '10.1.0', commit: ''},
+    {name: '10.1.0-2', commit: ''},
+    {name: '20.0.2', commit: ''},
+    {name: '100.0.0', commit: ''},
+    {name: '1000.0.0', commit: ''}
  ]

  const tagResolver = {
-    method: "non-existing-method"
+    method: 'non-existing-method'
  }
-  const sorted = sortTags(tags, tagResolver).map(function (tag) {
-    return tag.name
-  }).join(",")
+  const sorted = sortTags(tags, tagResolver)
+    .map(function (tag) {
+      return tag.name
+    })
+    .join(',')

-  expect(sorted).toStrictEqual(`1000.0.0,100.0.0,20.0.2,10.1.0,10.1.0-2,10.0.0,2.0.0,1.0.0,1.0.0-a01,0.1.0,0.1.0-b01,0.0.1,0.0.1-rc01`)
+  expect(sorted).toStrictEqual(
+    `1000.0.0,100.0.0,20.0.2,10.1.0,10.1.0-2,10.0.0,2.0.0,1.0.0,1.0.0-a01,0.1.0,0.1.0-b01,0.0.1,0.0.1-rc01`
+  )
 })

-
 it('Should order tags alphabetical', async () => {
-  jest.setTimeout(180000)
-
  const tags: TagInfo[] = [
-    { name: "0.0.1", commit: "" },
-    { name: "0.0.1-rc01", commit: "" },
-    { name: "0.1.0-b01", commit: "" },
-    { name: "1.0.0", commit: "" },
-    { name: "a", commit: "" },
-    { name: "1.0.0-a01", commit: "" },
-    { name: "2.0.0", commit: "" },
-    { name: "10.0.0", commit: "" },
-    { name: "v1", commit: "" },
-    { name: "10.1.0", commit: "" },
-    { name: "10.1.0-2", commit: "" },
-    { name: "20.0.2", commit: "" },
-    { name: "1000.0.0", commit: "" },
+    {name: '0.0.1', commit: ''},
+    {name: '0.0.1-rc01', commit: ''},
+    {name: '0.1.0-b01', commit: ''},
+    {name: '1.0.0', commit: ''},
+    {name: 'a', commit: ''},
+    {name: '1.0.0-a01', commit: ''},
+    {name: '2.0.0', commit: ''},
+    {name: '10.0.0', commit: ''},
+    {name: 'v1', commit: ''},
+    {name: '10.1.0', commit: ''},
+    {name: '10.1.0-2', commit: ''},
+    {name: '20.0.2', commit: ''},
+    {name: '1000.0.0', commit: ''}
  ]

  const tagResolver = {
-    method: "sort"
+    method: 'sort'
  }
-  const sorted = sortTags(tags, tagResolver).map(function (tag) {
-    return tag.name
-  }).join(",")
+  const sorted = sortTags(tags, tagResolver)
+    .map(function (tag) {
+      return tag.name
+    })
+    .join(',')

-  expect(sorted).toStrictEqual(`a,20.0.2,2.0.0,1000.0.0,10.1.0,10.1.0-2,10.0.0,1.0.0,1.0.0-a01,v1,0.1.0-b01,0.0.1,0.0.1-rc01`)
-})
+  expect(sorted).toStrictEqual(
+    `a,20.0.2,2.0.0,1000.0.0,10.1.0,10.1.0-2,10.0.0,1.0.0,1.0.0-a01,v1,0.1.0-b01,0.0.1,0.0.1-rc01`
+  )
+})
@@ -512,6 +512,25 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.


+lru-cache
+ISC
+The ISC License
+
+Copyright (c) Isaac Z. Schlueter and Contributors
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
+IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
+
 moment
 MIT
 Copyright (c) JS Foundation and other contributors
@@ -655,3 +674,22 @@ ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
 WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
 ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
 IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
+
+yallist
+ISC
+The ISC License
+
+Copyright (c) Isaac Z. Schlueter and Contributors
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
+IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
@@ -35,24 +35,24 @@
    "@actions/core": "^1.2.6",
    "@actions/exec": "^1.0.4",
    "@actions/github": "^4.0.0",
-    "@octokit/rest": "^18.0.9",
+    "@octokit/rest": "^18.0.10",
    "@types/semver": "^7.3.4",
    "moment": "^2.29.1",
-    "semver": "^7.3.2"
+    "semver": "^7.3.4"
  },
  "devDependencies": {
-    "@types/jest": "^26.0.15",
-    "@types/node": "^14.14.7",
-    "@typescript-eslint/parser": "^4.7.0",
+    "@types/jest": "^26.0.16",
+    "@types/node": "^14.14.10",
+    "@typescript-eslint/parser": "^4.9.0",
    "@vercel/ncc": "^0.25.1",
-    "eslint": "^7.13.0",
+    "eslint": "^7.14.0",
    "eslint-plugin-github": "^4.1.1",
-    "eslint-plugin-jest": "^24.1.0",
+    "eslint-plugin-jest": "^24.1.3",
    "jest": "^24.9.0",
    "jest-circus": "^26.6.3",
    "js-yaml": "^3.14.0",
-    "prettier": "2.1.2",
-    "ts-jest": "^24.3.0",
-    "typescript": "^4.0.5"
+    "prettier": "2.2.1",
+    "ts-jest": "^26.4.4",
+    "typescript": "^4.1.2"
  }
 }
@@ -56,14 +56,17 @@ export class Commits {
    core.info(
      `ℹ️ Found ${commits.length} commits from the GitHub API for ${owner}/${repo}`
    )
-    return commits.map(commit => ({
-      sha: commit.sha,
-      summary: commit.commit.message.split('\n')[0],
-      message: commit.commit.message,
-      date: moment(commit.commit.committer.date),
-      author: commit.commit.author.name,
-      prNumber: undefined
-    }))
+
+    return commits
+      .filter(commit => commit.sha)
+      .map(commit => ({
+        sha: commit.sha || '',
+        summary: commit.commit.message.split('\n')[0],
+        message: commit.commit.message,
+        date: moment(commit.commit.committer?.date),
+        author: commit.commit.author?.name || '',
+        prNumber: undefined
+      }))
  }

  private sortCommits(commits: CommitInfo[]): CommitInfo[] {
@@ -39,22 +39,23 @@ export class PullRequests {
        title: pr.data.title,
        htmlURL: pr.data.html_url,
        mergedAt: moment(pr.data.merged_at),
-        mergeCommitSha: pr.data.merge_commit_sha,
-        author: pr.data.user.login,
+        mergeCommitSha: pr.data.merge_commit_sha || '',
+        author: pr.data.user?.login || '',
        repoName: pr.data.base.repo.full_name,
-        labels: pr.data.labels.map(function (label) {
-          return label.name
-        }),
-        milestone: pr.data.milestone?.title,
-        body: pr.data.body,
-        assignees: pr.data.assignees?.map(function (asignee) {
-          return asignee.login
-        }),
-        requestedReviewers: pr.data.requested_reviewers?.map(function (
-          reviewer
-        ) {
-          return reviewer.login
-        })
+        labels:
+          pr.data.labels?.map(function (label) {
+            return label.name || ''
+          }) || [],
+        milestone: pr.data.milestone?.title || '',
+        body: pr.data.body || '',
+        assignees:
+          pr.data.assignees?.map(function (asignee) {
+            return asignee?.login || ''
+          }) || [],
+        requestedReviewers:
+          pr.data.requested_reviewers?.map(function (reviewer) {
+            return reviewer?.login || ''
+          }) || []
      }
    } catch (e) {
      core.warning(
@@ -90,20 +91,23 @@ export class PullRequests {
          title: pr.title,
          htmlURL: pr.html_url,
          mergedAt: moment(pr.merged_at),
-          mergeCommitSha: pr.merge_commit_sha,
-          author: pr.user.login,
+          mergeCommitSha: pr.merge_commit_sha || '',
+          author: pr.user?.login || '',
          repoName: pr.base.repo.full_name,
-          labels: pr.labels?.map(function (label) {
-            return label.name
-          }),
-          milestone: pr.milestone?.title,
-          body: pr.body,
-          assignees: pr.assignees?.map(function (asignee) {
-            return asignee.login
-          }),
-          requestedReviewers: pr.requested_reviewers?.map(function (reviewer) {
-            return reviewer.login
-          })
+          labels:
+            pr.labels?.map(function (label) {
+              return label.name || ''
+            }) || [],
+          milestone: pr.milestone?.title || '',
+          body: pr.body || '',
+          assignees:
+            pr.assignees?.map(function (asignee) {
+              return asignee?.login || ''
+            }) || [],
+          requestedReviewers:
+            pr.requested_reviewers?.map(function (reviewer) {
+              return reviewer?.login || ''
+            }) || []
        })
      }