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

probot stale #66

Merged
merged 3 commits into from Apr 25, 2019
Merged

probot stale #66

merged 3 commits into from Apr 25, 2019

Conversation

ahegde3
Copy link
Contributor

@ahegde3 ahegde3 commented Apr 15, 2019

issue #64
I have created a .yml file with suitable configuration to remove stale issues.
I have limited it only to pull request so it wont check for opened issues.
After 15 days of inactivity the pull request becomes stale and after further 7 days it will be closed.

@nurdtechie98
Copy link
Owner

@inishchith what do you think ??

@inishchith
Copy link
Collaborator

@ahegde3 Can you please keep only the commits which are relevant to this PR?

@ahegde3
Copy link
Contributor Author

ahegde3 commented Apr 16, 2019

yeah.I will make the change

@ahegde3
Copy link
Contributor Author

ahegde3 commented Apr 16, 2019

I have updated commit history

Copy link
Collaborator

@inishchith inishchith left a comment

Choose a reason for hiding this comment

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

@ahegde3 Thanks for working on this.
Just a few more changes and this will be good to merge.
Also a thing about the file, we can get rid of the comments, and at the top add a link to usage section.

.github/stale.yml Outdated Show resolved Hide resolved
.github/stale.yml Outdated Show resolved Hide resolved
.github/stale.yml Outdated Show resolved Hide resolved
.github/stale.yml Outdated Show resolved Hide resolved
.github/stale.yml Outdated Show resolved Hide resolved
.github/stale.yml Outdated Show resolved Hide resolved
.github/stale.yml Outdated Show resolved Hide resolved
@ahegde3
Copy link
Contributor Author

ahegde3 commented Apr 23, 2019

I have done the asked changes

@inishchith
Copy link
Collaborator

@ahegde3 Please remove the comments and update the 1st line with the link to usage section of the Probot repository.

@ahegde3
Copy link
Contributor Author

ahegde3 commented Apr 23, 2019

Have updated as per request

Copy link
Collaborator

@inishchith inishchith left a comment

Choose a reason for hiding this comment

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

@ahegde3 Thanks for your contributions.
LGTM.

Pinging @nurdtechie98 @shivam1708

@NightWing1998 NightWing1998 merged commit ca948a6 into nurdtechie98:dev Apr 25, 2019
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.

None yet

4 participants