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

feat(stale-and-close): add new options to change the days before close #224

Merged
merged 13 commits into from
Jan 16, 2021

Conversation

C0ZEN
Copy link
Contributor

@C0ZEN C0ZEN commented Nov 22, 2020

Closes #214
Closes #223

@mattklein123
Copy link

Any chance of getting this reviewed and merged? We are migrating from probot and miss this feature. Thank you so much.

@alyssawilk
Copy link

ping! We've got a PR out to work around this but it would be lovely to have native support.

Copy link

@richardst89 richardst89 left a comment

Choose a reason for hiding this comment

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

Love it! When can we start using it?

@C0ZEN
Copy link
Contributor Author

C0ZEN commented Jan 15, 2021

@hross if you are not available to maintain it I am willing to FYI. Just need the rights to merge.

@hross
Copy link
Contributor

hross commented Jan 15, 2021

@C0ZEN this looks good to me and I will merge it once you rebase.

src/IssueProcessor.ts Outdated Show resolved Hide resolved
src/enums/issue-type.enum.ts Outdated Show resolved Hide resolved
@C0ZEN
Copy link
Contributor Author

C0ZEN commented Jan 15, 2021

I will handle the rebase and naming conventions tomorrow.

to avoid a breaking change and simplify the configuration the old options 'daysBeforeStale' and 'daysBeforePrClose' are kept and new options are available to override them with 'daysBeforeIssueStale', 'daysBeforePrStale', 'daysBeforeIssueClose' and 'daysBeforePrClose'
it was not configured properly for TypeScript
README.md Outdated Show resolved Hide resolved
it was configured to only work with ts and it was not working well to be honest
also now the lint scripts will also run prettier
@C0ZEN
Copy link
Contributor Author

C0ZEN commented Jan 16, 2021

@hross FYI it is ready IMO.
I included a bit more content regarding the linter because I faced some issues with the table formatting and the enum but it's all for the better IMO.
I created small commits so you can review them easily.

@hross hross merged commit 552e4c6 into actions:main Jan 16, 2021
@C0ZEN C0ZEN deleted the feature/diff-number-days branch January 16, 2021 13:38
nijel added a commit to nijel/stale that referenced this pull request Feb 4, 2021
When different stale period is used for issues and pull requests, this code uses wrong one.

This was probably missed in actions#224

Fixes actions#299
hross pushed a commit that referenced this pull request Feb 5, 2021
* Fixed should be stale condition

When different stale period is used for issues and pull requests, this code uses wrong one.

This was probably missed in #224

Fixes #299

* Add tests for #299

Co-authored-by: Geoffrey Testelin <geoffrey.testelin@gmail.com>
pllim added a commit to pllim/astropy that referenced this pull request Feb 11, 2021
pllim added a commit to pllim/astropy that referenced this pull request Mar 8, 2021
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.

The inputs schema is not respected Support different number of days for PRs and issues
5 participants