gaspersic/freeCodeCamp
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

refactor: removed setInterval logic

Browse Source
This commit is contained in:
Randell Dawson
2018-11-09 20:18:55 -08:00
committed by mrugesh mohapatra
parent aa47b16156
commit 5070a87dd5
1 changed files with 44 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

81
labelOpenPrs.js
View File

@ -13,14 +13,8 @@ const { addComment } = require('./addComment');
octokit.authenticate(octokitAuth); octokit.authenticate(octokitAuth);
const labelsAdder = (number, existingLabels, labelsToAdd, log) => { const rateLimiter = (delay) => {
const newLabels = Object.keys(labelsToAdd).filter(label => !existingLabels.includes(label)); return new Promise(resolve => setTimeout(() => resolve(true), delay));
if (newLabels.length) {
addLabels(number, newLabels, log);
}
else {
log.update(number, false);
}
}; };
const { PrProcessingLog } = require('./prProcessingLog'); const { PrProcessingLog } = require('./prProcessingLog');
@ -42,42 +36,55 @@ const prPropsToGet = ['number', 'labels', 'user'];
log.start(); log.start();
console.log('Starting labeling process...'); console.log('Starting labeling process...');
let count = 0; for (let count = 0; count < openPRs.length; count++) {
const maxCount = openPRs.length; let { number, labels, user: { login: username } } = openPRs[count];
const { data: prFiles } = await octokit.pullRequests.getFiles({ owner, repo, number });
log.add(number)
const labelsToAdd = {}; // holds potential labels to add based on file path
let interval = setInterval(async () => { const guideFolderErrorsComment = guideFolderChecks(prFiles, username);
if (count < maxCount ) { if (guideFolderErrorsComment) {
let { number, labels, user: {login: username} } = openPRs[count]; const result = await octokit.issues.createComment({ owner, repo, number, body: guideFolderErrorsComment })
const { data: prFiles } = await octokit.pullRequests.getFiles({ owner, repo, number }); .catch((err) => {
log.add(number) console.log(`PR #${number} had an error when trying to add a comment\n`);
const existingLabels = labels.map(({ name }) => name); console.log(err)
const labelsToAdd = {}; // holds potential labels to add based on file path });
prFiles.forEach(({ filename }) => { if (result) {
/* remove '/challenges' from filename so variable second (below) will be the language */ console.log(`PR #${number} successfully added a comment\n`);
const filenameReplacement = filename.replace(/^curriculum\/challenges\//, 'curriculum\/'); await rateLimiter(3000);
const regex = /^(docs|curriculum|guide)(?:\/)(arabic|chinese|portuguese|russian|spanish)?\/?/ }
const [ _, first, second ] = filenameReplacement.match(regex) || []; // need an array to pass to labelsAdder labelsToAdd['status: needs update'] = 1;
if (first && validLabels[first]) { labelsToAdd[validLabels[first]] = 1 } }
if (second && validLabels[second]) { labelsToAdd[validLabels[second]] = 1 }
const guideFolderErrorsComment = guideFolderChecks(filename, username); const existingLabels = labels.map(({ name }) => name);
if (guideFolderErrorsComment) { prFiles.forEach(({ filename }) => {
addComment(number, guideFolderErrorsComment); /* remove '/challenges' from filename so language variable hold the language */
labelsToAdd['status: needs update'] = 1; 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
labelsAdder(number, existingLabels, labelsToAdd, log); if (articleType && validLabels[articleType]) {
labelsToAdd[validLabels[articleType]] = 1
}
if (language && validLabels[language]) {
labelsToAdd[validLabels[language]] = 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) {
addLabels(number, newLabels, log);
await rateLimiter(3000);
} }
else { else {
clearInterval(interval); log.update(number, false);
interval = null;
log.export();
} }
if (count % 25 === 0) { log.export() } if (count % 25 === 0) {
count++; log.export()
}, 1000); }
}
} }
})() })()
.then(() => { .then(() => {