Преглед изворни кода

fix: listing required roles should NOT mention/tag the roles, just list the title. (#5363)

Signed-off-by: Erik Westra <e.s.westra.95@gmail.com>
Erik Westra пре 6 месеци
родитељ
комит
dc918689f9
1 измењених фајлова са 1 додато и 1 уклоњено
  1. 1 1
      .github/workflows/lgtm.yml

+ 1 - 1
.github/workflows/lgtm.yml

@@ -187,7 +187,7 @@ jobs:
 
 
           // Check if user has any required reviewer role
           // Check if user has any required reviewer role
           if (commenterReviewerRoles.size === 0) {
           if (commenterReviewerRoles.size === 0) {
-            const rolesList = Array.from(requiredReviewerRoles).join(', ');
+            const rolesList = Array.from(requiredReviewerRoles).map(role => role.replace(`@${organization}/`, '')).join(', ');
             await github.rest.issues.createComment({
             await github.rest.issues.createComment({
               owner, repo, issue_number: prNumber,
               owner, repo, issue_number: prNumber,
               body: `@${commenter} You must be a member of one of the required reviewer roles to use /lgtm.\n\nRequired roles for this PR: ${rolesList}`
               body: `@${commenter} You must be a member of one of the required reviewer roles to use /lgtm.\n\nRequired roles for this PR: ${rolesList}`