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

Update stale.yml #1697

Merged
merged 1 commit into from
Sep 6, 2023
Merged

Update stale.yml #1697

merged 1 commit into from
Sep 6, 2023

Conversation

exitcode0
Copy link
Contributor

@exitcode0 exitcode0 commented Sep 6, 2023

waiting-response should be removed if the author of an issue or PR responds on the PR

Tweak the time-to-stale and time-to-close
Total time-to-close remains the same (65 days)

  • Stale label will be applied sooner (60 Days -> 30 Days)
  • Users will have more time between Stale label being added and issue being closed (5 days -> 35 days)
    • this better accounts for people taking extended PTO from their day jobs which may mean they miss the notification from github of the Stale label being added to their issue

`waiting-response` should be removed if the author of an issue or PR responds on the PR

Tweak the time to stale and time to close
Total time to close remains the same
`Stale` label will be applied sooner
Users will have more time between `Stale` label being added (and them getting notification) and the issue being closed, this better accounts for people taking extended PTO from their day jobs
Copy link
Collaborator

@monde monde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it so.
Thanks for putting some thought and giving good suggestions here @exitcode0

@MikeMondragon-okta MikeMondragon-okta merged commit 30d4a3c into okta:master Sep 6, 2023
1 check passed
@exitcode0 exitcode0 deleted the patch-4 branch September 7, 2023 01:46
@monde monde mentioned this pull request Sep 7, 2023
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.

3 participants