|
46 | 46 | echo "pr_sha=${PR_SHA}" >> $GITHUB_OUTPUT |
47 | 47 | echo "PR Number: ${PR_NUMBER}" |
48 | 48 | echo "PR SHA: ${PR_SHA}" |
| 49 | + if ! [[ "$PR_NUMBER" =~ ^[0-9]+$ ]]; then |
| 50 | + echo "Invalid PR number: $PR_NUMBER" |
| 51 | + exit 1 |
| 52 | + fi |
49 | 53 |
|
50 | 54 | - name: Create GitHub check run |
51 | 55 | uses: LouisBrunner/checks-action@6b626ffbad7cc56fd58627f774b9067e6118af23 # v2.0.0 |
@@ -201,8 +205,10 @@ jobs: |
201 | 205 |
|
202 | 206 | - name: Update HTML title with PR number |
203 | 207 | if: always() |
| 208 | + env: |
| 209 | + PR_NUMBER: ${{ needs.check_build.outputs.pr_number }} |
204 | 210 | run: | |
205 | | - sed "/<title>/s/>.*</>Test outputs from PR #${{ needs.check_build.outputs.pr_number }} HTML</" \ |
| 211 | + sed "/<title>/s/>.*</>Test outputs from PR #${PR_NUMBER} HTML</" \ |
206 | 212 | -i atex-html/index.html |
207 | 213 |
|
208 | 214 | - name: Merge test results from all versions |
@@ -244,8 +250,8 @@ jobs: |
244 | 250 |
|
245 | 251 | git add . |
246 | 252 | git commit -m "Test outputs from PR #${PR_NUMBER}" |
247 | | - git tag PR${PR_NUMBER} |
248 | | - git push origin PR${PR_NUMBER} |
| 253 | + git tag "PR${PR_NUMBER}" |
| 254 | + git push origin "PR${PR_NUMBER}" |
249 | 255 |
|
250 | 256 | - name: Submit results to Testing Farm |
251 | 257 | if: always() |
@@ -295,7 +301,7 @@ jobs: |
295 | 301 | GH_TOKEN: ${{ secrets.ATEX_RESULTS_TF_REPO_TOKEN }} |
296 | 302 | PR_NUMBER: ${{ needs.check_build.outputs.pr_number }} |
297 | 303 | run: | |
298 | | - git push --delete origin PR${PR_NUMBER} |
| 304 | + git push --delete origin "PR${PR_NUMBER}" |
299 | 305 |
|
300 | 306 | - name: Update GitHub check run |
301 | 307 | if: always() |
|
0 commit comments