Skip to content

Commit 35539da

Browse files
MostCromulentclaude
andcommitted
Check out before downloading the review artifact
actions/checkout cleans the workspace, so running it after the artifact download and the resolve step wiped changed_files.txt and review-data, leaving the poster with nothing to post. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent bf13906 commit 35539da

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

.github/workflows/card-script-review.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,11 @@ jobs:
3737
# Only PR builds carry review data; push builds have nothing to comment on.
3838
if: github.event.workflow_run.event == 'pull_request'
3939
steps:
40+
# Check out first: actions/checkout cleans the workspace, so it must run
41+
# before the artifact download and the files the resolve step writes.
42+
- name: Check out base scripts and corpus
43+
uses: actions/checkout@v5
44+
4045
- name: Download review data from the build
4146
uses: actions/download-artifact@v4
4247
with:
@@ -85,10 +90,6 @@ jobs:
8590
fs.writeFileSync('diff_lines.json', JSON.stringify(addedLines));
8691
core.info(`${cards.length} changed card file(s)`);
8792
88-
- name: Check out base scripts and corpus
89-
if: steps.pr.outputs.pull_number != ''
90-
uses: actions/checkout@v5
91-
9293
- name: Set up Python
9394
if: steps.pr.outputs.pull_number != ''
9495
uses: actions/setup-python@v6

0 commit comments

Comments
 (0)