diff --git a/.github/workflows/restrict-pr-authors.yaml b/.github/workflows/restrict-pr-authors.yaml index f0eaad3d5..51a381dc8 100644 --- a/.github/workflows/restrict-pr-authors.yaml +++ b/.github/workflows/restrict-pr-authors.yaml @@ -40,24 +40,26 @@ jobs: with: github-token: ${{ secrets.PR_CLOSE_TOKEN }} script: | + // List of allowed users (all lowercase) const allowed = [ "expsa", "moutazmuhammad", "ronozoro", - "Abubaker-Altaib", + "abubaker-altaib", "altexp", "the5abir", "ahmadaking", "kchyounes19", "abdurrahman-saber" - ]; + ].map(u => u.toLowerCase()); const pr = context.payload.pull_request; - const prAuthor = pr.user.login; + const prAuthor = pr.user.login.toLowerCase(); const owner = context.repo.owner; const repo = context.repo.repo; core.info(`PR author: ${prAuthor}`); + core.info(`Allowed users: ${allowed.join(", ")}`); // Check PR author if (!allowed.includes(prAuthor)) { @@ -79,7 +81,11 @@ jobs: }); for (const commit of commitList.data) { - const commitAuthor = commit.author ? commit.author.login : null; + const commitAuthor = commit.author ? commit.author.login.toLowerCase() : null; + + if (!commit.author) { + core.warning(`Commit ${commit.sha} has no GitHub-linked author.`); + } if (commitAuthor && !allowed.includes(commitAuthor)) { core.error(`Unauthorized commit author: ${commitAuthor}. Closing PR...`);