diff --git a/src/index.mjs b/src/index.mjs index 2ae2f9d..5aa3c09 100644 --- a/src/index.mjs +++ b/src/index.mjs @@ -38,9 +38,18 @@ async function ghActionBot() { comments.map(async comment => { const pull = await getPull(repo, comment.number); - if (pull.state !== 'open') return false; // filter out any closed pulls - if (PULL_AUTHOR_FILTER && !pull.owner !== PULL_AUTHOR_FILTER) return false; // filter on owner - if (PULL_LABEL_FILTER && !pull.labels.includes(PULL_LABEL_FILTER)) return false; // filter on label + if (pull.state !== 'open') { + logger.debug(`SKIP #${pull.number}: state is ${pull.state}`); + return false; // filter out any closed pulls + } + if (PULL_AUTHOR_FILTER && !pull.owner !== PULL_AUTHOR_FILTER) { + logger.debug(`SKIP #${pull.number}: author is ${pull.owner}`); + return false; // filter on owner + } + if (PULL_LABEL_FILTER && !pull.labels.includes(PULL_LABEL_FILTER)) { + logger.debug(`SKIP #${pull.number}: labels are ${pull.labels.join(',')}`); + return false; // filter on label + } return { comment, pull }; }) @@ -51,18 +60,25 @@ async function ghActionBot() { const commit = await getCommits(repo, pull.number, true); const buildStatus = await getCommitStatus(repo, commit.sha); // do nothing if the build has not started, or is pending or successful - if (!buildStatus || buildStatus.state === 'pending' || buildStatus.state === 'success') + if (!buildStatus || buildStatus.state === 'pending' || buildStatus.state === 'success') { + logger.debug(`SKIP #${pull.number}: build status is ${buildStatus.state}`); return false; + } return { comment, pull, commit, buildStatus }; }) )).filter(Boolean); - logger.log(`Found ${records.length} outstanding failures`); + if (records.length) { + logger.log(`Found ${records.length} outstanding failures`); + } else { + logger.debug(`Found ${records.length} outstanding failures`); + } await Promise.all( records.map(async record => { logger.log(`Re-testing PR #${record.pull.number}`); const comment = await createComment(repo, records[0].pull.number, PULL_RETEST_BODY); + logger.debug(`Created comment id ${comment.id}`); await deleteComment(repo, comment.id); }) );