Skip to content

Commit 2739f74

Browse files
fix: missed null checks (#54)
1 parent 394c969 commit 2739f74

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/helpers.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,13 +122,13 @@ export async function afterCLA(app, claSignatureInfo) {
122122
order: "desc"
123123
});
124124

125-
const filteredPrs = prs?.filter(pr => pr.user.login === githubUsername);
126-
console.log(`Found ${filteredPrs?.length} open PRs for ${githubUsername} in ${org}:`, filteredPrs?.map(pr => pr.number).join(', '));
125+
const filteredPrs = prs?.filter(pr => pr?.user?.login === githubUsername);
126+
console.log(`Found ${filteredPrs?.length} open PRs for ${githubUsername} in ${org}:`, filteredPrs?.map(pr => pr?.number).join(', '));
127127
let failuresToRemoveLabel = 0;
128128
for (const pr of filteredPrs) {
129129
const { owner, repo } = parseRepoUrl(pr?.repository_url) || {};
130130
const hasPendingCLALabel = pr.labels?.some(label => label?.name?.toLowerCase() === "pending cla");
131-
console.log(`PR #${pr.number} has "Pending CLA" label: ${hasPendingCLALabel}`);
131+
console.log(`PR #${pr?.number} has "Pending CLA" label: ${hasPendingCLALabel}`);
132132
if (hasPendingCLALabel) {
133133
try {
134134
await removePendingCLALabel(octokit, owner, repo, pr?.number);

0 commit comments

Comments
 (0)