fix: changed rateLimter time in close-open

This commit is contained in:
Randell Dawson
2018-11-29 12:11:54 -08:00
committed by mrugesh mohapatra
parent f443f2bbd6
commit 22bfbe95b9
3 changed files with 8 additions and 47 deletions

View File

@ -1,39 +0,0 @@
const { validLabels } = require('./validation');
const { addLabels } = require('./pr-tasks');
const { rateLimiter } = require('./utils');
const labeler = async (number, prFiles, currentLabels, guideFolderErrorsComment) => {
const labelsToAdd = {}; // holds potential labels to add based on file path
if (guideFolderErrorsComment) {
labelsToAdd['status: needs update'] = 1;
}
const existingLabels = currentLabels.map(({ name }) => name);
prFiles.forEach(({ filename }) => {
/* remove '/challenges' from filename so language variable hold the language */
const filenameReplacement = filename.replace(/^curriculum\/challenges\//, 'curriculum\/');
const regex = /^(docs|curriculum|guide)(?:\/)(arabic|chinese|portuguese|russian|spanish)?\/?/
const [ _, articleType, language ] = filenameReplacement.match(regex) || []; // need an array to pass to labelsAdder
if (articleType && validLabels[articleType]) {
labelsToAdd[validLabels[articleType]] = 1
}
if (language && validLabels[language]) {
labelsToAdd[validLabels[language]] = 1
}
if (articleType === 'curriculum') {
labelsToAdd['status: need to test locally'] = 1;
}
})
/* this next section only adds needed labels which are NOT currently on the PR. */
const newLabels = Object.keys(labelsToAdd).filter(label => !existingLabels.includes(label));
if (newLabels.length) {
if (process.env.PRODUCTION_RUN === 'true') {
addLabels(number, newLabels);
}
await rateLimiter(+process.env.RATELIMIT_INTERVAL | 1500);
}
return newLabels;
};
module.exports = { labeler };

View File

@ -29,16 +29,16 @@ const getUserInput = async () => {
const lastPR = prs[prs.length - 1].number;
log.setFirstLast({ firstPR, lastPR });
for (let { number, data: { errorDesc } } of prs) {
//if (errorDesc !== "unknown error") {
if (errorDesc !== "unknown error") {
log.add(number, { closedOpened: true, errorDesc });
if (process.env.PRODUCTION_RUN === 'true') {
await closeOpen(number);
await rateLimiter(2000);
await rateLimiter(120000);
}
//}
//else {
// log.add(number, { closedOpened: false, errorDesc });
//}
}
else {
log.add(number, { closedOpened: false, errorDesc });
}
}
})
.then(() => {

View File

@ -11,11 +11,11 @@ octokit.authenticate(octokitAuth);
const closeOpen = async (number) => {
const result = await octokit.pullRequests.update({ owner, repo , number, state: 'closed', base: 'master' })
.then(async () => {
await rateLimiter(10000);
await rateLimiter(5000);
return octokit.pullRequests.update({ owner, repo , number, state: 'open', base: 'master' })
})
.then(async () => {
await rateLimiter(2000);
await rateLimiter(1000);
await addComment(number, 'Closed and Reopened this PR to attempt to resolve a specific Travis build failure.');
})
.catch(async (err) => {