actions/stale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge back multi issue changes and allow for injection of issues

Browse Source
This commit is contained in:
Ross Brodbeck 2020-04-16 10:10:14 -04:00
parent d7719d70c7
commit dbb0a7494e
2 changed files with 21 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/IssueProcessor.ts
View File

@ -13,9 +13,9 @@ export interface IssueProcessorOptions {
daysBeforeStale: number;
daysBeforeClose: number;
staleIssueLabel: string;
exemptIssueLabel: string;
exemptIssueLabels: string;
stalePrLabel: string;
exemptPrLabel: string;
exemptPrLabels: string;
onlyLabels: string;
operationsPerRun: number;
debugOnly: boolean;
@ -35,7 +35,10 @@ export class IssueProcessor {
this.client = new github.GitHub(options.repoToken);
}
async processIssues(page: number = 1): Promise<number> {
async processIssues(
page: number = 1,
getIssues: (page: number) => Promise<IssueList> = this.getIssues // used for injecting issues to test
): Promise<number> {
if (this.options.debugOnly) {
core.warning(
'Executing in debug mode. Debug output will be written but no issues will be processed.'
@ -48,7 +51,7 @@ export class IssueProcessor {
}
// get the next batch of issues
const issues: IssueList = await this.getIssues(page);
const issues: IssueList = await getIssues(page);
if (issues.data.length <= 0) {
core.debug('No more issues found to process. Exiting.');
@ -69,9 +72,9 @@ export class IssueProcessor {
const staleLabel: string = isPr
? this.options.stalePrLabel
: this.options.staleIssueLabel;
const exemptLabel: string = isPr
? this.options.exemptPrLabel
: this.options.exemptIssueLabel;
const exemptLabels = IssueProcessor.parseCommaSeparatedString(
isPr ? this.options.exemptPrLabels : this.options.exemptIssueLabels
);
const issueType: string = isPr ? 'pr' : 'issue';
if (!staleMessage) {
@ -79,7 +82,7 @@ export class IssueProcessor {
continue;
}
if (exemptLabel && IssueProcessor.isLabeled(issue, exemptLabel)) {
if (exemptLabels.some((exemptLabel: string) => IssueProcessor.isLabeled(issue, exemptLabel))) {
core.debug(`Skipping ${issueType} because it has an exempt label`);
continue; // don't process exempt issues
}
@ -115,7 +118,7 @@ export class IssueProcessor {
}
// do the next batch
return await this.processIssues(page + 1);
return this.processIssues(page + 1);
}
// grab issues from github in baches of 100
@ -187,4 +190,11 @@ export class IssueProcessor {
new Date().getTime() - new Date(issue.updated_at).getTime();
return millisSinceLastUpdated >= daysInMillis;
}
private static parseCommaSeparatedString(s: string): string[] {
// String.prototype.split defaults to [''] when called on an empty string
// In this case, we'd prefer to just return an empty array indicating no labels
if (!s.length) return [];
return s.split(',');
}
}

4
src/main.ts
View File

@ -25,9 +25,9 @@ function getAndValidateArgs(): IssueProcessorOptions {
core.getInput('days-before-close', {required: true})
),
staleIssueLabel: core.getInput('stale-issue-label', {required: true}),
exemptIssueLabel: core.getInput('exempt-issue-label'),
exemptIssueLabels: core.getInput('exempt-issue-labels'),
stalePrLabel: core.getInput('stale-pr-label', {required: true}),
exemptPrLabel: core.getInput('exempt-pr-label'),
exemptPrLabels: core.getInput('exempt-pr-labels'),
onlyLabels: core.getInput('only-labels'),
operationsPerRun: parseInt(
core.getInput('operations-per-run', {required: true})