Skip to content

Commit d9eda70

Browse files
minipuftclaude
andcommitted
fix: correct GitHub Actions job status variable usage in PR validation
- Fix incorrect usage of process.env.GITHUB_JOB_STATUS (doesn't exist) - Use ${{ job.status }} directly as intended in github-script action - Resolves potential undefined variable issue in comment generation - Ensures proper success/failure status detection for PR comments This fix addresses a JavaScript runtime error in the comment generation step where an undefined environment variable was being used as fallback. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent a82c55d commit d9eda70

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/pr-validation.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ jobs:
183183
console.log(`Warning: Could not read changed_files.txt: ${e.message}`);
184184
changedFiles = 'Unable to read changed files (this is normal for some PR types)';
185185
}
186-
const status = process.env.GITHUB_JOB_STATUS || '${{ job.status }}';
186+
const status = '${{ job.status }}';
187187
const runUrl = `${context.payload.repository.html_url}/actions/runs/${context.runId}`;
188188
let message;
189189
if (status === 'success') {

0 commit comments

Comments
 (0)