feat: add debugging output to script
This commit is contained in:
@@ -38,9 +38,18 @@ async function ghActionBot() {
|
|||||||
comments.map(async comment => {
|
comments.map(async comment => {
|
||||||
const pull = await getPull(repo, comment.number);
|
const pull = await getPull(repo, comment.number);
|
||||||
|
|
||||||
if (pull.state !== 'open') return false; // filter out any closed pulls
|
if (pull.state !== 'open') {
|
||||||
if (PULL_AUTHOR_FILTER && !pull.owner !== PULL_AUTHOR_FILTER) return false; // filter on owner
|
logger.debug(`SKIP #${pull.number}: state is ${pull.state}`);
|
||||||
if (PULL_LABEL_FILTER && !pull.labels.includes(PULL_LABEL_FILTER)) return false; // filter on label
|
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 };
|
return { comment, pull };
|
||||||
})
|
})
|
||||||
@@ -51,18 +60,25 @@ async function ghActionBot() {
|
|||||||
const commit = await getCommits(repo, pull.number, true);
|
const commit = await getCommits(repo, pull.number, true);
|
||||||
const buildStatus = await getCommitStatus(repo, commit.sha);
|
const buildStatus = await getCommitStatus(repo, commit.sha);
|
||||||
// do nothing if the build has not started, or is pending or successful
|
// 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 false;
|
||||||
|
}
|
||||||
return { comment, pull, commit, buildStatus };
|
return { comment, pull, commit, buildStatus };
|
||||||
})
|
})
|
||||||
)).filter(Boolean);
|
)).filter(Boolean);
|
||||||
|
|
||||||
|
if (records.length) {
|
||||||
logger.log(`Found ${records.length} outstanding failures`);
|
logger.log(`Found ${records.length} outstanding failures`);
|
||||||
|
} else {
|
||||||
|
logger.debug(`Found ${records.length} outstanding failures`);
|
||||||
|
}
|
||||||
|
|
||||||
await Promise.all(
|
await Promise.all(
|
||||||
records.map(async record => {
|
records.map(async record => {
|
||||||
logger.log(`Re-testing PR #${record.pull.number}`);
|
logger.log(`Re-testing PR #${record.pull.number}`);
|
||||||
const comment = await createComment(repo, records[0].pull.number, PULL_RETEST_BODY);
|
const comment = await createComment(repo, records[0].pull.number, PULL_RETEST_BODY);
|
||||||
|
logger.debug(`Created comment id ${comment.id}`);
|
||||||
await deleteComment(repo, comment.id);
|
await deleteComment(repo, comment.id);
|
||||||
})
|
})
|
||||||
);
|
);
|
||||||
|
|||||||
Reference in New Issue
Block a user