From 47abaa81dc4bf6e77a6c9e23869eeb0f73d5504b Mon Sep 17 00:00:00 2001 From: Felix Schneider <99918022+trueberryless@users.noreply.github.com> Date: Mon, 23 Mar 2026 16:53:18 +0100 Subject: [PATCH 1/3] fix: make message parameters optional --- src/main.ts | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/main.ts b/src/main.ts index 7107bcc..0d859b5 100644 --- a/src/main.ts +++ b/src/main.ts @@ -28,10 +28,13 @@ export async function run() { ) // Get the action inputs. - const issueMessage: string = core.getInput('issue_message', { - required: true - }) - const prMessage: string = core.getInput('pr_message', { required: true }) + const issueMessage: string = core.getInput('issue_message') + if (isIssue && !issueMessage) + return core.info('Skipping... No issue message configured') + + const prMessage: string = core.getInput('pr_message') + if (!isIssue && !prMessage) + return core.info('Skipping... No PR message configured') const octokit = new Octokit({ auth: core.getInput('repo_token', { required: true }) From 5a8f1847ede96dc9a3d4091fa145581775509dfa Mon Sep 17 00:00:00 2001 From: Felix Schneider <99918022+trueberryless@users.noreply.github.com> Date: Mon, 23 Mar 2026 17:29:16 +0100 Subject: [PATCH 2/3] refactor: use isPullRequest for better readability --- src/main.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main.ts b/src/main.ts index 0d859b5..3d0df68 100644 --- a/src/main.ts +++ b/src/main.ts @@ -33,7 +33,7 @@ export async function run() { return core.info('Skipping... No issue message configured') const prMessage: string = core.getInput('pr_message') - if (!isIssue && !prMessage) + if (isPullRequest && !prMessage) return core.info('Skipping... No PR message configured') const octokit = new Octokit({ From 81986edb37b6739c768f97347031dafbf4235d0e Mon Sep 17 00:00:00 2001 From: Felix Schneider <99918022+trueberryless@users.noreply.github.com> Date: Mon, 23 Mar 2026 17:34:01 +0100 Subject: [PATCH 3/3] test: add test cases --- __tests__/main.test.ts | 34 ++++++++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts index 37e0445..64cc6e7 100644 --- a/__tests__/main.test.ts +++ b/__tests__/main.test.ts @@ -290,5 +290,39 @@ describe('main.ts', () => { expect(result).toBe(false) }) + + it('Skips if is issue and no issue_message is provided', async () => { + github.context.payload.issue = { number: 10 } + github.context.payload.pull_request = undefined as any + + core.getInput + .mockReset() + .mockReturnValueOnce('') + .mockReturnValueOnce('PR_MESSAGE') + + await main.run() + + expect(core.info).toHaveBeenCalledWith( + 'Skipping... No issue message configured' + ) + expect(mocktokit.paginate).not.toHaveBeenCalled() + }) + + it('Skips if is PR and no pr_message is provided', async () => { + github.context.payload.issue = undefined as any + github.context.payload.pull_request = { number: 10 } + + core.getInput + .mockReset() + .mockReturnValueOnce('ISSUE_MESSAGE') + .mockReturnValueOnce('') + + await main.run() + + expect(core.info).toHaveBeenCalledWith( + 'Skipping... No PR message configured' + ) + expect(mocktokit.paginate).not.toHaveBeenCalled() + }) }) })