fix(validate-pr): set skipped output for admin/maintain authors (#158)

Copilot · jpnurmi · web-flow · commit 6c8116fd012a · 2026-03-28T14:07:30.000+01:00
* Initial plan * fix: add missing setOutput('skipped', 'true') for maintainer check Agent-Logs-Url: https://github.com/getsentry/github-workflows/sessions/679d4d30-c819-45cb-b992-04eb4dbc6193 Co-authored-by: jpnurmi <140617+jpnurmi@users.noreply.github.com> * fix(validate-pr): set skipped output for admin/maintain authors Co-authored-by: jpnurmi <140617+jpnurmi@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: jpnurmi <140617+jpnurmi@users.noreply.github.com>
diff --git a/validate-pr/scripts/validate-pr.js b/validate-pr/scripts/validate-pr.js
@@ -60,6 +60,7 @@ module.exports = async ({ github, context, core }) => {
   const authorIsMaintainer = await isMaintainer(repo.owner, repo.repo, prAuthor);
   if (authorIsMaintainer) {
     core.info(`PR author ${prAuthor} has admin/maintain access. Skipping.`);
+    core.setOutput('skipped', 'true');
     return;
   }
   core.info(`PR author ${prAuthor} is not a maintainer.`);

Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,7 @@ module.exports = async ({ github, context, core }) => {`
`60`	`60`	`const authorIsMaintainer = await isMaintainer(repo.owner, repo.repo, prAuthor);`
`61`	`61`	`if (authorIsMaintainer) {`
`62`	`62`	core.info(`PR author ${prAuthor} has admin/maintain access. Skipping.`);
	`63`	`+ core.setOutput('skipped', 'true');`
`63`	`64`	`return;`
`64`	`65`	`}`
`65`	`66`	core.info(`PR author ${prAuthor} is not a maintainer.`);