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

Remove skip-stale-issue-message and skip-stale-pr-message options #455

Closed
C0ZEN opened this issue May 23, 2021 · 0 comments · Fixed by #457
Closed

Remove skip-stale-issue-message and skip-stale-pr-message options #455

C0ZEN opened this issue May 23, 2021 · 0 comments · Fixed by #457
Labels
bug Something isn't working

Comments

@C0ZEN
Copy link
Contributor

C0ZEN commented May 23, 2021

Describe your issue

These options seems useless to me.
We can simply define for example that no message will be sent if stale-issue-message is empty.
Also, it's strange that there is no option to cover the close message.

I can provide a PR.

@C0ZEN C0ZEN added the bug Something isn't working label May 23, 2021
C0ZEN added a commit to C0ZEN/stale that referenced this issue May 23, 2021
Closes actions#405
Closes actions#455

BREAKING CHANGES: remove skip-stale-issue-message and skip-stale-pr-message options. If you used this option, replace it by an empty message for the options stale-issue-message and stale-pr-message
C0ZEN added a commit to C0ZEN/stale that referenced this issue May 25, 2021
Closes actions#405
Closes actions#455

BREAKING CHANGES: remove skip-stale-issue-message and skip-stale-pr-message options. If you used this option, replace it by an empty message for the options stale-issue-message and stale-pr-message
luketomlinson pushed a commit that referenced this issue May 25, 2021
…#457)

* feat(options): simplify config by removing skip stale message options

Closes #405
Closes #455

BREAKING CHANGES: remove skip-stale-issue-message and skip-stale-pr-message options. If you used this option, replace it by an empty message for the options stale-issue-message and stale-pr-message

* build(dist): update dist

also lint some files

* docs(readme): update the docs by removing the skip options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant