|
33 | 33 | python-version: 3.9 |
34 | 34 |
|
35 | 35 |
|
36 | | - - name: Fecth branch |
| 36 | + - name: Fetch branch |
37 | 37 | run: git fetch origin ${{ github.event.pull_request.base.ref }} |
38 | 38 |
|
39 | 39 | - name: Collect information |
@@ -64,49 +64,50 @@ jobs: |
64 | 64 | - name: Run "preReview" script |
65 | 65 | run: | |
66 | 66 | PR_NUMBER=${{ github.event.pull_request.number }} |
| 67 | + BASE_REPO=${{ github.event.pull_request.base.repo.full_name }} |
67 | 68 | REVIEW_MESSAGE=$(python pre_review.py "$CHANGED_FILES" "$CONTRIBUTOR") |
68 | 69 | echo "REVIEW_MESSAGE<<EOF" >> $GITHUB_ENV |
69 | 70 | echo "$REVIEW_MESSAGE" >> $GITHUB_ENV |
70 | 71 | echo "EOF" >> $GITHUB_ENV |
71 | 72 |
|
72 | | - # Save to env |
73 | | - echo "REVIEW_MESSAGE<<EOF" >> $GITHUB_ENV |
74 | | - echo "$REVIEW_MESSAGE" >> $GITHUB_ENV |
75 | | - echo "EOF" >> $GITHUB_ENV |
76 | | - |
77 | | - # Print to log explicitly |
78 | 73 | echo "===== REVIEW_MESSAGE =====" |
79 | 74 | echo "$REVIEW_MESSAGE" |
80 | | - echo "==========================" |
| 75 | + echo "===========================" |
81 | 76 | |
82 | 77 | # Attempt to comment (will skip for forks) |
83 | | - gh pr comment "$PR_NUMBER" --body "$REVIEW_MESSAGE" -R "${{ github.event.pull_request.base.repo.full_name }}" |
84 | | -
|
85 | | - # Assign invalid label |
86 | | - if echo "$REVIEW_MESSAGE" | grep -Eqi 'Missing required contribution:|Incorrect changes:'; then |
87 | | - gh pr edit "$PR_NUMBER" --add-label "invalid" -R "$BASE_REPO" |
| 78 | + gh pr comment "$PR_NUMBER" --body "$REVIEW_MESSAGE" -R "$BASE_REPO" |
| 79 | +
|
| 80 | + # -------------------------------------------------------------------------- |
| 81 | + # LABELS |
| 82 | + # -------------------------------------------------------------------------- |
| 83 | + # --- "Changes Requested" or "Awaiting Maintainer Validation" labels --- |
| 84 | + if echo "$REVIEW_MESSAGE" | grep -Eq '\- \[ \]'; then |
| 85 | + gh pr edit "$PR_NUMBER" --add-label "Changes Requested" -R "$BASE_REPO" |
| 86 | + gh pr edit "$PR_NUMBER" --remove-label "Awaiting Maintainer Validation" -R "$BASE_REPO" |
88 | 87 | else |
89 | | - gh pr edit "$PR_NUMBER" --remove-label "invalid" -R "$BASE_REPO" |
| 88 | + gh pr edit "$PR_NUMBER" --add-label "Awaiting Maintainer Validation" -R "$BASE_REPO" |
| 89 | + gh pr edit "$PR_NUMBER" --remove-label "Changes Requested" -R "$BASE_REPO" |
90 | 90 | fi |
91 | 91 |
|
92 | | - # Assign changes requested label |
93 | | - if echo "$REVIEW_MESSAGE" | grep -Eqi 'Contributors.md'; then |
94 | | - gh pr edit "$PR_NUMBER" --add-label "changes requested" -R "$BASE_REPO" |
| 92 | +
|
| 93 | + # --- Invalid label --- |
| 94 | + if echo "$REVIEW_MESSAGE" | grep -qi 'Missing required contribution'; then |
| 95 | + gh pr edit "$PR_NUMBER" --add-label "invalid" -R "$BASE_REPO" |
95 | 96 | else |
96 | | - gh pr edit "$PR_NUMBER" --remove-label "changes requested" -R "$BASE_REPO" |
| 97 | + gh pr edit "$PR_NUMBER" --remove-label "invalid" -R "$BASE_REPO" |
97 | 98 | fi |
98 | 99 |
|
99 | | - # Assign Awaiting Maintainer Validation if there are checkboxes |
100 | | - if echo "$REVIEW_MESSAGE" | grep -q '\- \[ \]'; then |
101 | | - gh pr edit "$PR_NUMBER" --add-label "Awaiting Maintainer Validation" -R "$BASE_REPO" |
| 100 | +
|
| 101 | + # --- Conflict present label --- |
| 102 | + if echo "$REVIEW_MESSAGE" | grep -qi 'archived'; then |
| 103 | + gh pr edit "$PR_NUMBER" --add-label "Conflict present" -R "$BASE_REPO" |
102 | 104 | else |
103 | | - gh pr edit "$PR_NUMBER" --remove-label "Awaiting Maintainer Validation" -R "$BASE_REPO" |
| 105 | + gh pr edit "$PR_NUMBER" --remove-label "Conflict present" -R "$BASE_REPO" |
104 | 106 | fi |
105 | 107 |
|
106 | 108 | env: |
107 | 109 | CHANGED_FILES: ${{ env.CHANGED_FILES }} |
108 | 110 | CONTRIBUTOR: ${{ env.CONTRIBUTOR }} |
109 | | - PR_NUMBERS: ${{ env.GITHUB_TOKEN }} |
110 | 111 | GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
111 | 112 |
|
112 | 113 | remove-awaiting-on-merge: |
|
0 commit comments