-
Notifications
You must be signed in to change notification settings - Fork 366
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
feat(options): add new option ignore-updates to stale even with activity #540
Conversation
d366c75
to
3cfdcda
Compare
@luketomlinson any news? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good for the most part @C0ZEN. My only question is how we name the new options. And while I think these particular new options provide some value, I really want to start limiting the number of new options we add.
Any update on this? 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @C0ZEN 🥇
9eef113
to
f05bb6f
Compare
instead of using a real linter with auto fix and the tests verbose as fuck
bc015ce
to
5cd25d2
Compare
@luketomlinson ready |
Changes
remove-issue-stale-when-updated
andremove-pr-stale-when-updated
. Fixes Unable to comprehend handling of stale issue when days-before-issue-stale is -1 #451ignore-updates
with issues/PRs override to stale based on the creation date instead of the update date to allow staling sooner and disregard any kind of activity to alter the process. Resolve Include option to ignore PR activity #435