From 4420b2cdd28dd7ac77185beb37ad824d9d33f561 Mon Sep 17 00:00:00 2001 From: Anika Maskara Date: Wed, 25 Jun 2025 16:50:17 -0400 Subject: [PATCH] add context to approval workflow --- .github/workflows/review.yml | 11 +++++++++++ lib/post-review-status.js | 5 +++-- lib/post-review-status.js.map | 2 +- post-review-status/action.yml | 3 +++ src/post-review-status.ts | 5 +++-- 5 files changed, 21 insertions(+), 5 deletions(-) diff --git a/.github/workflows/review.yml b/.github/workflows/review.yml index 2ecba2f..88ea157 100644 --- a/.github/workflows/review.yml +++ b/.github/workflows/review.yml @@ -27,3 +27,14 @@ jobs: pull-request: ${{ github.event.pull_request.number }} owner: ${{ github.repository_owner }} repo: github-actions + test-post-status-check-feature-branch: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v2 + - id: security-review + uses: ./post-review-status + with: + pull-request: ${{ github.event.pull_request.number }} + owner: ${{ github.repository_owner }} + repo: github-actions + context: github-actions/important-feature-branch diff --git a/lib/post-review-status.js b/lib/post-review-status.js index e5c7291..7c6e469 100644 --- a/lib/post-review-status.js +++ b/lib/post-review-status.js @@ -28,6 +28,7 @@ async function main() { const github = github_1.getOctokit(token, {}); const owner = core.getInput("owner"); const repo = core.getInput("repo"); + const contextName = core.getInput("context") || github_1.context.repo.repo; let pr_num = core.getInput("pull-request"); if (pr_num == "") pr_num = github_1.context.payload.pull_request.head.ref.split("/")[2]; @@ -69,8 +70,8 @@ async function main() { sha: pr.head.sha, state: status, target_url: `https://github.com/${github_1.context.repo.owner}/${github_1.context.repo.repo}/pull/${github_1.context.payload.pull_request.number}`, - description: `status of ${github_1.context.repo.owner}/${github_1.context.repo.repo}#${github_1.context.payload.pull_request.number}`, - context: `${github_1.context.repo.repo}/review`, + description: `status of ${github_1.context.repo.owner}/${contextName}#${github_1.context.payload.pull_request.number}`, + context: `${contextName}/review`, }); } // eslint-disable-next-line @typescript-eslint/no-explicit-any diff --git a/lib/post-review-status.js.map b/lib/post-review-status.js.map index a168d41..c2263e8 100644 --- a/lib/post-review-status.js.map +++ b/lib/post-review-status.js.map @@ -1 +1 @@ -{"version":3,"file":"post-review-status.js","sourceRoot":"","sources":["../src/post-review-status.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;AAAA,oDAAsC;AACtC,4CAAsD;AAEtD,OAAO,CAAC,EAAE,CAAC,oBAAoB,EAAE,WAAW,CAAC,CAAC;AAC9C,IAAI,EAAE,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;AAE1B,KAAK,UAAU,IAAI;IACjB,MAAM,KAAK,GAAG,IAAI,CAAC,QAAQ,CAAC,cAAc,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;IAChE,MAAM,MAAM,GAAG,mBAAU,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IAErC,MAAM,KAAK,GAAG,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;IACrC,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;IAEnC,IAAI,MAAM,GAAG,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC;IAC3C,IAAI,MAAM,IAAI,EAAE;QACd,MAAM,GAAG,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAW,CAAC;IAE1E,MAAM,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,MAAM,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC;QAC/C,KAAK,EAAE,KAAK;QACZ,IAAI,EAAE,IAAI;QACV,WAAW,EAAE,CAAC,MAAM;KACrB,CAAC,CAAC;IAEH,IAAI,MAAM,GAAgD,SAAS,CAAC;IACpE,IACE,gBAAO,CAAC,SAAS,KAAK,qBAAqB;QAC3C,gBAAO,CAAC,OAAO,CAAC,MAAM,KAAK,WAAW,EACtC;QACA,IAAI,gBAAO,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,KAAK,UAAU,EAAE;YAC/C,MAAM,GAAG,SAAS,CAAC;SACpB;aAAM,IAAI,gBAAO,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,KAAK,mBAAmB,EAAE;YAC/D,MAAM,GAAG,SAAS,CAAC;SACpB;KACF;SAAM;QACL,MAAM,KAAK,GAAG,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,OAAO,CAAC;QACpD,MAAM,KAAK,GAAG;wBACM,KAAK;;;;;UAKnB,CAAC;QACP,MAAM,EACJ,IAAI,EAAE,EAAE,cAAc,EAAE,MAAM,EAAE,GACjC,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;QACpC,IAAI,MAAM,KAAK,UAAU,EAAE;YACzB,MAAM,GAAG,SAAS,CAAC;SACpB;aAAM,IAAI,MAAM,KAAK,mBAAmB,EAAE;YACzC,MAAM,GAAG,SAAS,CAAC;SACpB;KACF;IACD,OAAO,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,kBAAkB,CAAC;QAC1C,KAAK,EAAE,KAAK;QACZ,IAAI,EAAE,IAAI;QACV,GAAG,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG;QAChB,KAAK,EAAE,MAAM;QACb,UAAU,EAAE,sBAAsB,gBAAO,CAAC,IAAI,CAAC,KAAK,IAClD,gBAAO,CAAC,IAAI,CAAC,IACf,SAAS,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,MAAM,EAAE;QAC/C,WAAW,EAAE,aAAa,gBAAO,CAAC,IAAI,CAAC,KAAK,IAAI,gBAAO,CAAC,IAAI,CAAC,IAAI,IAC/D,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,MAChC,EAAE;QACF,OAAO,EAAE,GAAG,gBAAO,CAAC,IAAI,CAAC,IAAI,SAAS;KACvC,CAAC,CAAC;AACL,CAAC;AAED,8DAA8D;AAC9D,SAAS,WAAW,CAAC,GAAQ;IAC3B,OAAO,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IACnB,IAAI,CAAC,SAAS,CAAC,oBAAoB,GAAG,EAAE,CAAC,CAAC;AAC5C,CAAC"} \ No newline at end of file +{"version":3,"file":"post-review-status.js","sourceRoot":"","sources":["../src/post-review-status.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;AAAA,oDAAsC;AACtC,4CAAsD;AAEtD,OAAO,CAAC,EAAE,CAAC,oBAAoB,EAAE,WAAW,CAAC,CAAC;AAC9C,IAAI,EAAE,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;AAE1B,KAAK,UAAU,IAAI;IACjB,MAAM,KAAK,GAAG,IAAI,CAAC,QAAQ,CAAC,cAAc,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;IAChE,MAAM,MAAM,GAAG,mBAAU,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IAErC,MAAM,KAAK,GAAG,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;IACrC,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;IACnC,MAAM,WAAW,GAAG,IAAI,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,gBAAO,CAAC,IAAI,CAAC,IAAI,CAAC;IAElE,IAAI,MAAM,GAAG,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC;IAC3C,IAAI,MAAM,IAAI,EAAE;QACd,MAAM,GAAG,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAW,CAAC;IAE1E,MAAM,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,MAAM,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC;QAC/C,KAAK,EAAE,KAAK;QACZ,IAAI,EAAE,IAAI;QACV,WAAW,EAAE,CAAC,MAAM;KACrB,CAAC,CAAC;IAEH,IAAI,MAAM,GAAgD,SAAS,CAAC;IACpE,IACE,gBAAO,CAAC,SAAS,KAAK,qBAAqB;QAC3C,gBAAO,CAAC,OAAO,CAAC,MAAM,KAAK,WAAW,EACtC;QACA,IAAI,gBAAO,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,KAAK,UAAU,EAAE;YAC/C,MAAM,GAAG,SAAS,CAAC;SACpB;aAAM,IAAI,gBAAO,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,KAAK,mBAAmB,EAAE;YAC/D,MAAM,GAAG,SAAS,CAAC;SACpB;KACF;SAAM;QACL,MAAM,KAAK,GAAG,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,OAAO,CAAC;QACpD,MAAM,KAAK,GAAG;wBACM,KAAK;;;;;UAKnB,CAAC;QACP,MAAM,EACJ,IAAI,EAAE,EAAE,cAAc,EAAE,MAAM,EAAE,GACjC,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;QACpC,IAAI,MAAM,KAAK,UAAU,EAAE;YACzB,MAAM,GAAG,SAAS,CAAC;SACpB;aAAM,IAAI,MAAM,KAAK,mBAAmB,EAAE;YACzC,MAAM,GAAG,SAAS,CAAC;SACpB;KACF;IACD,OAAO,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,kBAAkB,CAAC;QAC1C,KAAK,EAAE,KAAK;QACZ,IAAI,EAAE,IAAI;QACV,GAAG,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG;QAChB,KAAK,EAAE,MAAM;QACb,UAAU,EAAE,sBAAsB,gBAAO,CAAC,IAAI,CAAC,KAAK,IAClD,gBAAO,CAAC,IAAI,CAAC,IACf,SAAS,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,MAAM,EAAE;QAC/C,WAAW,EAAE,aAAa,gBAAO,CAAC,IAAI,CAAC,KAAK,IAAI,WAAW,IACzD,gBAAO,CAAC,OAAO,CAAC,YAAa,CAAC,MAChC,EAAE;QACF,OAAO,EAAE,GAAG,WAAW,SAAS;KACjC,CAAC,CAAC;AACL,CAAC;AAED,8DAA8D;AAC9D,SAAS,WAAW,CAAC,GAAQ;IAC3B,OAAO,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IACnB,IAAI,CAAC,SAAS,CAAC,oBAAoB,GAAG,EAAE,CAAC,CAAC;AAC5C,CAAC"} \ No newline at end of file diff --git a/post-review-status/action.yml b/post-review-status/action.yml index 947c6db..5bcdbdf 100644 --- a/post-review-status/action.yml +++ b/post-review-status/action.yml @@ -16,6 +16,9 @@ inputs: pull-request: description: Pull-request number required: false + context: + description: Name of the status check. If none is provided, `{repository-name/review}` will be used + required: false outputs: result: description: "Commit status details" diff --git a/src/post-review-status.ts b/src/post-review-status.ts index e10c919..cf6a9a5 100644 --- a/src/post-review-status.ts +++ b/src/post-review-status.ts @@ -10,6 +10,7 @@ async function main(): Promise { const owner = core.getInput("owner"); const repo = core.getInput("repo"); + const contextName = core.getInput("context") || context.repo.repo; let pr_num = core.getInput("pull-request"); if (pr_num == "") @@ -57,10 +58,10 @@ async function main(): Promise { target_url: `https://github.com/${context.repo.owner}/${ context.repo.repo }/pull/${context.payload.pull_request!.number}`, - description: `status of ${context.repo.owner}/${context.repo.repo}#${ + description: `status of ${context.repo.owner}/${contextName}#${ context.payload.pull_request!.number }`, - context: `${context.repo.repo}/review`, + context: `${contextName}/review`, }); }