Compare commits

..

11 Commits

Author SHA1 Message Date
Thomas Hughes
1a8e37e7ad Update README to latest release version 2019-11-01 13:43:25 +01:00
Jeremy Epling
08796943dd Merge pull request #14 from actions/kaylangan-per-day
Example runs once per day instead of per hour
2019-08-29 14:52:07 -04:00
Kayla Ngan
ded3b10c18 Example runs once per day instead of per hour 2019-08-29 14:50:13 -04:00
Ryan Wilson-Perkin
9f69ef12ad Feature: Exempt labels from action (#11)
* Change name to isLabeled for re-use with "exempt"

* Add args for exempt labels

* Skip operation if labeled exempt

* Only check for exempt label if defined

Since the exempt label parameter is not required, we don't want to both
checking for it if it was not defined.

* Don't reduce operationsLeft if exempt

This variable just tracks the number of operations we're using to avoid
rate limiting. Since the exempt check wouldn't result in a rate-limited
request, we don't need to reduce the variable.

* Update the action.yml with new exempt values

* Update README with example of exempt labels

* Remove redundant defaults
2019-08-27 15:58:14 -04:00
Edward Thomson
cc7f3505fc Spelling fixes in action description (#7)
Fix typo 'iusses' -> 'issues'.  Also update 'prs' to 'pull requests'.
2019-08-19 14:34:16 -04:00
John Briggs
0cad222e9d Fix wasLastUpdatedBefore to be in days, not hours (#8)
The parameters and variables names make it seem like this is the intended behavior.
2019-08-19 14:28:41 -04:00
Shawn Napora
903ff7344d Merge pull request #6 from actions/case-insensitive-bugfix
bugfix for case insensitive issues
2019-08-07 10:42:01 -04:00
Shawn Napora
898b0bc63f format, IssueLabels -> IssueLabel 2019-08-07 10:36:13 -04:00
Danny McCormick
4924c684b5 More nits 2019-08-07 10:33:00 -04:00
Danny McCormick
c7694b0cea Packag nits (#5) 2019-08-07 10:19:42 -04:00
Shawn Napora
1acc1d62d9 bugfix for case insensitive issues 2019-08-07 10:14:20 -04:00
4 changed files with 38 additions and 28 deletions

View File

@@ -11,13 +11,13 @@ Basic:
name: "Close stale issues"
on:
schedule:
- cron: "0 * * * *"
- cron: "0 0 * * *"
jobs:
stale:
runs-on: ubuntu-latest
steps:
- uses: actions/stale@v1
- uses: actions/stale@v1.1.0
with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
stale-issue-message: 'Message to comment on stale issues. If none provided, will not mark issues stale'
@@ -29,13 +29,13 @@ Configure stale timeouts:
name: "Close stale issues"
on:
schedule:
- cron: "0 * * * *"
- cron: "0 0 * * *"
jobs:
stale:
runs-on: ubuntu-latest
steps:
- uses: actions/stale@v1
- uses: actions/stale@v1.1.0
with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
stale-issue-message: 'This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days'
@@ -48,17 +48,19 @@ Configure labels:
name: "Close stale issues"
on:
schedule:
- cron: "0 * * * *"
- cron: "0 0 * * *"
jobs:
stale:
runs-on: ubuntu-latest
steps:
- uses: actions/stale@v1
- uses: actions/stale@v1.1.0
with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
stale-issue-message: 'Stale issue message'
stale-pr-message: 'Stale issue message'
stale-issue-label: 'no-issue-activity'
exempt-issue-label: 'awaiting-approval'
stale-pr-label: 'no-pr-activity'
exempt-pr-label: 'awaiting-approval'
```

View File

@@ -6,24 +6,28 @@ inputs:
description: 'Token for the repo. Can be passed in using {{ secrets.GITHUB_TOKEN }}'
required: true
stale-issue-message:
description: 'The message to post on the issue when tagging it. If none provided, will not mark iusses stale.'
description: 'The message to post on the issue when tagging it. If none provided, will not mark issues stale.'
stale-pr-message:
description: 'The message to post on the pr when tagging it. If none provided, will not mark prs stale.'
description: 'The message to post on the pr when tagging it. If none provided, will not mark pull requests stale.'
days-before-stale:
description: 'The number of days old an issue can be before marking it stale'
default: 60
days-before-close:
description: 'The number of days to wait to close an issue or pr after it being marked stale'
description: 'The number of days to wait to close an issue or pull request after it being marked stale'
default: 7
stale-issue-label:
description: 'The label to apply when an issue is stale'
default: 'Stale'
exempt-issue-label:
description: 'The label to apply when an issue is exempt from being marked stale'
stale-pr-label:
description: 'The label to apply when a pr is stale'
description: 'The label to apply when a pull request is stale'
default: 'Stale'
exempt-pr-label:
description: 'The label to apply when a pull request is exempt from being marked stale'
operations-per-run:
description: 'The maximum number of operations per run, used to control rate limiting'
default: 30
runs:
using: 'node12'
main: 'lib/main.js'
main: 'lib/main.js'

View File

@@ -12,12 +12,12 @@
},
"repository": {
"type": "git",
"url": "git+https://github.com/actions/start-vm-action.git"
"url": "git+https://github.com/actions/stale.git"
},
"keywords": [
"actions",
"node",
"setup"
"stale"
],
"author": "GitHub",
"license": "MIT",

View File

@@ -2,6 +2,9 @@ import * as core from '@actions/core';
import * as github from '@actions/github';
import * as Octokit from '@octokit/rest';
type Issue = Octokit.IssuesListForRepoResponseItem;
type IssueLabel = Octokit.IssuesListForRepoResponseItemLabelsItem;
type Args = {
repoToken: string;
staleIssueMessage: string;
@@ -9,7 +12,9 @@ type Args = {
daysBeforeStale: number;
daysBeforeClose: number;
staleIssueLabel: string;
exemptIssueLabel: string;
stalePrLabel: string;
exemptPrLabel: string;
operationsPerRun: number;
};
@@ -51,13 +56,16 @@ async function processIssues(
let staleMessage = isPr ? args.stalePrMessage : args.staleIssueMessage;
if (!staleMessage) {
core.debug(`skipping ${isPr ? "pr" : "issue"} due to empty message`);
core.debug(`skipping ${isPr ? 'pr' : 'issue'} due to empty message`);
continue;
}
let staleLabel = isPr ? args.stalePrLabel : args.staleIssueLabel;
let exemptLabel = isPr ? args.exemptPrLabel : args.exemptIssueLabel;
if (isLabeledStale(issue, staleLabel)) {
if (exemptLabel && isLabeled(issue, exemptLabel)) {
continue;
} else if (isLabeled(issue, staleLabel)) {
if (wasLastUpdatedBefore(issue, args.daysBeforeClose)) {
operationsLeft -= await closeIssue(client, issue);
} else {
@@ -82,20 +90,14 @@ async function processIssues(
return await processIssues(client, args, operationsLeft, page + 1);
}
function isLabeledStale(
issue: Octokit.IssuesListForRepoResponseItem,
label: string
): boolean {
const labelComparer = l =>
label.localeCompare(l.name, undefined, {sensitivity: 'accent'});
function isLabeled(issue: Issue, label: string): boolean {
const labelComparer: (l: IssueLabel) => boolean = l =>
label.localeCompare(l.name, undefined, {sensitivity: 'accent'}) === 0;
return issue.labels.filter(labelComparer).length > 0;
}
function wasLastUpdatedBefore(
issue: Octokit.IssuesListForRepoResponseItem,
num_days: number
): boolean {
const daysInMillis = 1000 * 60 * 60 * num_days;
function wasLastUpdatedBefore(issue: Issue, num_days: number): boolean {
const daysInMillis = 1000 * 60 * 60 * 24 * num_days;
const millisSinceLastUpdated =
new Date().getTime() - new Date(issue.updated_at).getTime();
return millisSinceLastUpdated >= daysInMillis;
@@ -103,7 +105,7 @@ function wasLastUpdatedBefore(
async function markStale(
client: github.GitHub,
issue: Octokit.IssuesListForRepoResponseItem,
issue: Issue,
staleMessage: string,
staleLabel: string
): Promise<number> {
@@ -128,7 +130,7 @@ async function markStale(
async function closeIssue(
client: github.GitHub,
issue: Octokit.IssuesListForRepoResponseItem
issue: Issue
): Promise<number> {
core.debug(`closing issue ${issue.title} for being stale`);
@@ -154,7 +156,9 @@ function getAndValidateArgs(): Args {
core.getInput('days-before-close', {required: true})
),
staleIssueLabel: core.getInput('stale-issue-label', {required: true}),
exemptIssueLabel: core.getInput('exempt-issue-label'),
stalePrLabel: core.getInput('stale-pr-label', {required: true}),
exemptPrLabel: core.getInput('exempt-pr-label'),
operationsPerRun: parseInt(
core.getInput('operations-per-run', {required: true})
)