Skip to content

Commit

Permalink
ci: correct PR number retrieval to fix size-report action (vuejs#11223)
Browse files Browse the repository at this point in the history
Co-authored-by: 三咲智子 Kevin Deng <[email protected]>
  • Loading branch information
jh-leong and sxzz authored Jun 28, 2024
1 parent f2acd51 commit 2f11a45
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 5 deletions.
10 changes: 10 additions & 0 deletions .github/workflows/size-data.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,3 +41,13 @@ jobs:
with:
name: size-data
path: temp/size

- name: Save PR number
if: ${{github.event_name == 'pull_request'}}
run: echo ${{ github.event.number }} > ./pr.txt

- uses: actions/upload-artifact@v4
if: ${{github.event_name == 'pull_request'}}
with:
name: pr-number
path: pr.txt
31 changes: 26 additions & 5 deletions .github/workflows/size-report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,19 @@ jobs:
- name: Install dependencies
run: pnpm install

- name: Download PR number
uses: dawidd6/action-download-artifact@v3
with:
name: pr-number
run_id: ${{ github.event.workflow_run.id }}
path: /tmp/pr-number

- name: Read PR Number
id: pr-number
uses: juliangruber/read-file-action@v1
with:
path: /tmp/pr-number/pr.txt

- name: Download Size Data
uses: dawidd6/action-download-artifact@v6
with:
Expand All @@ -55,10 +68,18 @@ jobs:
- name: Prepare report
run: pnpm tsx scripts/size-report.ts > size-report.md

- name: Read Size Report
id: size-report
uses: juliangruber/read-file-action@v1
with:
path: ./size-report.md

- name: Create Comment
uses: thollander/actions-[email protected]
uses: actions-cool/maintain-one-comment@v3
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
filePath: size-report.md
pr_number: ${{ github.event.workflow_run.pull_requests[0].number }}
comment_tag: VUE_CORE_SIZE
token: ${{ secrets.GITHUB_TOKEN }}
number: ${{ steps.pr-number.outputs.content }}
body: |
${{ steps.size-report.outputs.content }}
<!-- VUE_CORE_SIZE -->
body-include: '<!-- VUE_CORE_SIZE -->'

0 comments on commit 2f11a45

Please sign in to comment.