fix: Simplify Plugin Live Check Reporting
This commit is contained in:
@@ -1,7 +1,7 @@
|
|||||||
name: Plugin Live Check
|
name: Plugin Live Check
|
||||||
|
|
||||||
on:
|
on:
|
||||||
pull_request_target:
|
pull_request:
|
||||||
branches: [master]
|
branches: [master]
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
inputs:
|
inputs:
|
||||||
@@ -11,39 +11,26 @@ on:
|
|||||||
type: string
|
type: string
|
||||||
|
|
||||||
concurrency:
|
concurrency:
|
||||||
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
|
group: ${{ github.workflow }}-${{ github.ref }}
|
||||||
cancel-in-progress: true
|
cancel-in-progress: true
|
||||||
|
|
||||||
permissions: {}
|
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
live-check:
|
live-check:
|
||||||
name: Plugin Live Check
|
name: Plugin Live Check
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|
||||||
outputs:
|
|
||||||
should_comment: ${{ steps.comment-artifact.outputs.should_comment }}
|
|
||||||
|
|
||||||
permissions:
|
permissions:
|
||||||
contents: read
|
contents: read
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
- name: Checkout PR Merge Commit
|
|
||||||
if: github.event_name == 'pull_request_target'
|
|
||||||
uses: actions/checkout@v4
|
|
||||||
with:
|
|
||||||
ref: refs/pull/${{ github.event.pull_request.number }}/merge
|
|
||||||
fetch-depth: 0
|
|
||||||
|
|
||||||
- name: Checkout Repository
|
- name: Checkout Repository
|
||||||
if: github.event_name == 'workflow_dispatch'
|
|
||||||
uses: actions/checkout@v4
|
uses: actions/checkout@v4
|
||||||
with:
|
with:
|
||||||
fetch-depth: 0
|
fetch-depth: 0
|
||||||
|
|
||||||
- name: Get Changed Plugin Files
|
- name: Get Changed Plugin Files
|
||||||
id: changed-files
|
id: changed-files
|
||||||
if: github.event_name == 'pull_request_target'
|
if: github.event_name == 'pull_request'
|
||||||
uses: tj-actions/changed-files@v45
|
uses: tj-actions/changed-files@v45
|
||||||
with:
|
with:
|
||||||
files: |
|
files: |
|
||||||
@@ -67,9 +54,7 @@ jobs:
|
|||||||
uses: actions/setup-node@v4
|
uses: actions/setup-node@v4
|
||||||
with:
|
with:
|
||||||
node-version: '20'
|
node-version: '20'
|
||||||
# Avoid writing contributor-controlled caches in the base branch's
|
cache: 'npm'
|
||||||
# cache scope when running under pull_request_target.
|
|
||||||
cache: ${{ github.event_name == 'workflow_dispatch' && 'npm' || '' }}
|
|
||||||
|
|
||||||
- name: Install Dependencies
|
- name: Install Dependencies
|
||||||
if: steps.targets.outputs.files != ''
|
if: steps.targets.outputs.files != ''
|
||||||
@@ -86,50 +71,10 @@ jobs:
|
|||||||
node scripts/live-check-plugin.js ${{ steps.targets.outputs.files }} > live-check-output.txt 2>&1
|
node scripts/live-check-plugin.js ${{ steps.targets.outputs.files }} > live-check-output.txt 2>&1
|
||||||
echo "exit_code=$?" >> "$GITHUB_OUTPUT"
|
echo "exit_code=$?" >> "$GITHUB_OUTPUT"
|
||||||
cat live-check-output.txt
|
cat live-check-output.txt
|
||||||
|
cat live-check-output.txt >> "$GITHUB_STEP_SUMMARY"
|
||||||
- name: Prepare PR Comment Artifact
|
|
||||||
id: comment-artifact
|
|
||||||
if: always() && github.event_name == 'pull_request_target'
|
|
||||||
run: |
|
|
||||||
if [ "${{ steps.changed-files.outputs.any_changed }}" = 'true' ] && [ -f live-check-output.txt ]; then
|
|
||||||
echo 'should_comment=true' >> "$GITHUB_OUTPUT"
|
|
||||||
else
|
|
||||||
echo 'should_comment=false' >> "$GITHUB_OUTPUT"
|
|
||||||
fi
|
|
||||||
|
|
||||||
- name: Upload PR Comment Artifact
|
|
||||||
if: always() && steps.comment-artifact.outputs.should_comment == 'true'
|
|
||||||
uses: actions/upload-artifact@v4
|
|
||||||
with:
|
|
||||||
name: plugin-live-check-comment
|
|
||||||
path: live-check-output.txt
|
|
||||||
retention-days: 1
|
|
||||||
|
|
||||||
- name: Fail On Real Errors
|
- name: Fail On Real Errors
|
||||||
if: steps.targets.outputs.files != '' && steps.live-check.outputs.exit_code != '0'
|
if: steps.targets.outputs.files != '' && steps.live-check.outputs.exit_code != '0'
|
||||||
run: |
|
run: |
|
||||||
echo "Live check reported at least one FAIL — see the job log or PR comment above."
|
echo "Live check reported at least one FAIL — see the job log or workflow summary."
|
||||||
exit 1
|
exit 1
|
||||||
|
|
||||||
comment:
|
|
||||||
name: Post PR Comment
|
|
||||||
needs: live-check
|
|
||||||
if: always() && github.event_name == 'pull_request_target' && needs.live-check.outputs.should_comment == 'true'
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
|
|
||||||
permissions:
|
|
||||||
actions: read
|
|
||||||
pull-requests: write
|
|
||||||
|
|
||||||
steps:
|
|
||||||
- name: Download PR Comment Artifact
|
|
||||||
uses: actions/download-artifact@v4
|
|
||||||
with:
|
|
||||||
name: plugin-live-check-comment
|
|
||||||
|
|
||||||
- name: Post PR Comment
|
|
||||||
uses: marocchino/sticky-pull-request-comment@v3
|
|
||||||
with:
|
|
||||||
number: ${{ github.event.pull_request.number }}
|
|
||||||
header: plugin-live-check
|
|
||||||
path: live-check-output.txt
|
|
||||||
|
|||||||
Reference in New Issue
Block a user