Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow for issues/PRs to be closed after manually being marked as stale #103

Merged
merged 1 commit into from
Jul 13, 2020

Conversation

phillipj
Copy link
Contributor

@phillipj phillipj commented Jul 8, 2020

Previously this action would stop and skip all issues or pull requests it found if no stale-issue-message | stale-pr-message option had been configured.

In practise that meant this action could not be used in repositories where the maintainers would want to manually mark issues and pull requests as stale, instead of this action being responsible for the said marking.

That's what these changes allows for, maintainers being responsible for marking, but still want help from this action to close those issues or pull requests X days after they got manually marked as stale by labelling them.

Configuring days-before-stale: -1 will activate that behaviour, inspired by how days-before-close: -1 works already.

Would that sound like a good fit for this action by any chance?

Previously this action would stop and skip all issues or pull requests
it found if no `stale-issue-message | stale-pr-message` option had
been configured.

In practise that meant this action could not be used in repositories
where the maintainers would want to manually mark issues and pull
requests as stale, instead of this action being responsible for the
said marking.

That's what these changes allows for, maintainers being responsible
for marking, but still want help from this action to close those
issues or pull requests X days after they got manually marked as
stale by labelling them.

Configuring `days-before-stale: -1` will activate that behaviour.
@phillipj
Copy link
Contributor Author

phillipj commented Jul 8, 2020

Forgot to cross link this with nodejs/github-bot#261 where we've discussed using this action to close issues and pull requests in nodejs/node.

@hross hross merged commit f111c4f into actions:master Jul 13, 2020
@phillipj phillipj deleted the configurable-mark-as-stale branch July 18, 2020 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants