2 Commits

Author SHA1 Message Date
aeb0368508 fix: hide debugging output when value is '0' 2019-02-19 16:50:17 -07:00
106ed777fa feat: check for pending status after comment
keep re-trying until state changes to pending, up to 3 times. if state does not change, remove from history so another run will retry
2019-02-19 16:42:21 -07:00
2 changed files with 22 additions and 2 deletions

View File

@@ -6,11 +6,14 @@ import getPull from './lib/get_pull.mjs';
import getCommits from './lib/get_commits.mjs';
import getCommitStatus from './lib/get_commit_status.mjs';
import createPullComment from './lib/create_pull_comment.mjs';
import execAndCheck from './lib/exec_and_check.mjs';
import History from './lib/history.mjs';
// load env vars from .env file
dotenv.config();
const sleep = async ms => new Promise(resolve => setTimeout(resolve, ms));
async function ghActionBot() {
// parse repo name from cli and create repo instance
const repo = createRepo(process.argv.splice(2)[0]);
@@ -22,6 +25,7 @@ async function ghActionBot() {
PULL_LABEL_FILTER,
PULL_AUTHOR_FILTER,
PULL_RETEST_BODY,
ACTION_CHECK_DELAY = 3000,
} = process.env;
// load the history module
@@ -90,7 +94,23 @@ async function ghActionBot() {
await Promise.all(
records.map(record =>
createPullComment(repo, record.pull.number, PULL_RETEST_BODY, { delete: true })
execAndCheck(
() => createPullComment(repo, record.pull.number, PULL_RETEST_BODY, { delete: true }),
async () => {
logger.debug(`CHECK #${record.pull.number}: verify state change`);
// wait for the ci to restart
await sleep(ACTION_CHECK_DELAY);
// check that the commit status is now pending
const buildStatus = await getCommitStatus(repo, record.commit.sha);
return buildStatus.state === 'pending';
},
() => {
history.remove(record.comment);
logger.error(`Pull comment failed to trigger action on pull #${record.pull.number}`);
}
)
)
);
}

View File

@@ -35,7 +35,7 @@ const logger = {
console.error(wrapMessage(args));
},
debug(...args) {
if (!process.env.DEBUG) return;
if (process.env.DEBUG === '0' || !process.env.DEBUG) return;
console.log(wrapMessage(args));
},
};