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

set stale daysUntilClose to a really big number #2162

Merged
merged 1 commit into from
Dec 18, 2017

Conversation

jspricke
Copy link
Member

No description provided.

@jspricke jspricke mentioned this pull request Dec 18, 2017
@SergioRAgostinho SergioRAgostinho merged commit 9298940 into PointCloudLibrary:master Dec 18, 2017
@jspricke jspricke deleted the stale_fix branch December 18, 2017 19:58
@@ -3,7 +3,8 @@
# Number of days of inactivity before an Issue or Pull Request becomes stale
daysUntilStale: 60
# Number of days of inactivity before a stale Issue or Pull Request is closed
# daysUntilClose: 7
# 2**32-1 until https://github.com/probot/stale/issues/79 is fixed
daysUntilClose: 4294967295

Choose a reason for hiding this comment

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

Heads up that this is causing an error and preventing stale from running against this repository: RangeError: errorInvalid time value

Copy link
Member

@SergioRAgostinho SergioRAgostinho Dec 21, 2017

Choose a reason for hiding this comment

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

Thanks for reporting. What's the biggest number we can put here?

Edit: We basically need a workaround to prevent the issues/PRs from being closed.

Copy link
Member

Choose a reason for hiding this comment

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

Just saw you suggested a number in the original post.

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.

4 participants