summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Nate Moore <natemoo-re@users.noreply.github.com> 2023-05-31 17:15:14 -0500
committerGravatar GitHub <noreply@github.com> 2023-05-31 17:15:14 -0500
commitc4c086e5e70bdd347e1c7601212a04b4f523578e (patch)
treefafd74fd0200894df3c08bcfe5925c12e4780cfd
parente0ca0d8c8a01ad73b711a9639b854b3f52900750 (diff)
downloadastro-c4c086e5e70bdd347e1c7601212a04b4f523578e.tar.gz
astro-c4c086e5e70bdd347e1c7601212a04b4f523578e.tar.zst
astro-c4c086e5e70bdd347e1c7601212a04b4f523578e.zip
Fix `semver minor` action (#7261)
* Update check-merge.yml * chore: update action * chore(action): better check if block exists * chore(action): update check-merge action
-rw-r--r--.github/workflows/check-merge.yml74
1 files changed, 36 insertions, 38 deletions
diff --git a/.github/workflows/check-merge.yml b/.github/workflows/check-merge.yml
index e31c09bf4..990d1f846 100644
--- a/.github/workflows/check-merge.yml
+++ b/.github/workflows/check-merge.yml
@@ -1,10 +1,12 @@
name: Check mergeability
-on: pull_request # run on pull request events
+on: pull_request
permissions:
- # grant write permission on the pull-requests endpoint
pull-requests: write
+ checks: write
+ statuses: write
+
jobs:
check:
runs-on: ubuntu-latest
@@ -12,15 +14,18 @@ jobs:
- name: Check if there is already a block on this PR
id: set-blocks
uses: actions/github-script@v6
+ env:
+ issue_number: ${{ github.event.number }}
with:
script: |
- const url = 'https://api.github.com/repos/' + context.repo.owner + '/' + context.repo.repo + '/pulls/' + context.issue.number + '/reviews';
- console.log('reviews URL', url);
- const result = await github.request(url);
- console.log(result);
- const reviews = result.data;
- for(const review of reviews) {
- if(review.user.id === 41898282 && review.state === 'CHANGES_REQUESTED') {
+ const { data: reviews } = await github.rest.pulls.listReviews({
+ owner: context.repo.owner,
+ repo: context.repo.repo,
+ pull_number: process.env.issue_number,
+ });
+ console.log(reviews);
+ for (const review of reviews) {
+ if (review.user.id === 41898282 && review.state === 'CHANGES_REQUESTED') {
return 'block';
}
}
@@ -54,38 +59,31 @@ jobs:
done
- name: Add label
+ uses: actions/github-script@v6
if: steps.find-blockers.outputs.found == 'true'
- run: |
- curl --request POST \
- --url https://api.github.com/repos/${{github.repository}}/issues/${{github.event.number}}/labels \
- --header 'authorization: Bearer ${{ secrets.GITHUB_TOKEN }}' \
- --header 'content-type: application/json' \
- -d '["semver minor"]'
-
-
- - name: Find Comment
- uses: peter-evans/find-comment@v2
- id: fc
+ env:
+ issue_number: ${{ github.event.number }}
with:
- issue-number: ${{ github.event.number }}
- comment-author: 'github-actions[bot]'
+ script: |
+ github.rest.issues.addLabels({
+ issue_number: process.env.issue_number,
+ owner: context.repo.owner,
+ repo: context.repo.repo,
+ labels: ['semver minor']
+ });
- - name: Send PR review
+ - name: Change PR Status
+ uses: actions/github-script@v6
if: steps.find-blockers.outputs.found == 'true'
- uses: peter-evans/create-or-update-comment@v3
- continue-on-error: true
+ env:
+ issue_number: ${{ github.event.number }}
with:
- comment-id: ${{ steps.fc.outputs.comment-id }}
- issue-number: ${{ github.event.number }}
- body: |
- This PR is blocked because it contains a `minor` changeset. A reviewer will merge this at the next release if approved.
- edit-mode: replace
- - name: Change PR status
- if: steps.find-blockers.outputs.found == 'true'
- run: |
- curl --request POST \
- --url https://api.github.com/repos/${{github.repository}}/pulls/${{github.event.number}}/reviews \
- --header 'authorization: Bearer ${{ secrets.GITHUB_TOKEN }}' \
- --header 'content-type: application/json' \
- -d '{"event":"REQUEST_CHANGES", body: ""}'
+ script: |
+ github.rest.pulls.createReview({
+ owner: context.repo.owner,
+ repo: context.repo.repo,
+ pull_number: process.env.issue_number,
+ event: 'REQUEST_CHANGES',
+ body: 'This PR is blocked because it contains a `minor` changeset. A reviewer will merge this at the next release if approved.'
+ });