aboutsummaryrefslogtreecommitdiffstats
path: root/.github/workflows
diff options
context:
space:
mode:
authorSteven Arcangeli <stevearc@stevearc.com>2023-12-04 10:18:11 -0800
committerSteven Arcangeli <stevearc@stevearc.com>2023-12-04 10:18:11 -0800
commit96cb2d5e3d8d7c927af2c86a11a71ee6d09d9031 (patch)
treecfdad83f22af9f4f515cefb990ac68fde20b7bed /.github/workflows
parent9fb74bc74ea0955699534cd68267ed9528f18e4e (diff)
ci: attempt to fix the changes requested comment flow
Diffstat (limited to '.github/workflows')
-rw-r--r--.github/workflows/automation_request_changes_message.yml16
1 files changed, 11 insertions, 5 deletions
diff --git a/.github/workflows/automation_request_changes_message.yml b/.github/workflows/automation_request_changes_message.yml
index 4484853..352391e 100644
--- a/.github/workflows/automation_request_changes_message.yml
+++ b/.github/workflows/automation_request_changes_message.yml
@@ -4,11 +4,11 @@ on:
types: [submitted]
jobs:
- comment:
+ check:
runs-on: ubuntu-latest
steps:
- name: Check Previous Reviews
- id: check
+ id: result
uses: actions/github-script@v7
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
@@ -22,16 +22,22 @@ jobs:
});
const changesRequestedReviews = reviews.data.filter((review) => review.state === 'CHANGES_REQUESTED');
const firstTimeChangesRequested = changesRequestedReviews.length === 1;
- return firstTimeChangesRequested;
+ // Set firstTimeChangesRequested as output of this step
+ core.setOutput("firstTimeChangesRequested", firstTimeChangesRequested)
+ comment:
+ needs: check
+ runs-on: ubuntu-latest
+ steps:
- name: Comment on PR
uses: actions/github-script@v7
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
- const runPreviousCheckStep = steps.check.outputs.result
const pr = context.payload.pull_request;
- if (runPreviousCheckStep === 'true') {
+ // Get output from previous 'check' step
+ const firstTimeChangesRequested = needs.check.outputs.firstTimeChangesRequested
+ if (firstTimeChangesRequested === 'true') {
github.rest.issues.createComment({
owner: context.repo.owner,
repo: context.repo.repo,