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(remove-stale-when-updated): add 2 options for issues and prs #383

Merged
merged 26 commits into from
Apr 30, 2021

Conversation

C0ZEN
Copy link
Contributor

@C0ZEN C0ZEN commented Mar 9, 2021

Changes:

  • Highlight more values in the logs
  • Enhance some logs to make them more humanized
  • Stop the action sooner right after it was un-stale to avoid trying to close the issue
  • Add 2 new options remove-issue-stale-when-updated and remove-pr-stale-when-updated

Closes #377

C0ZEN and others added 23 commits November 22, 2020 10:55
feat(stale-and-close): add new options to change the days before clos…
Remove the stale label when labeled with an exempt one  (actions#268)
feat(statistics): display some stats in the logs (actions#337)
* docs(only-labels): remove duplicated option and improve descriptions

a bad rebase happend

* docs(readme): use a multi-line array and remove the optional column

the option column was not helpful since each value is optional
the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width

* style(readme): break line for the statistics

* docs(readme): add a better description for the ascending option

* docs(action): add missing punctuation
Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2.
- [Release notes](https://github.com/octokit/rest.js/releases)
- [Commits](octokit/rest.js@v18.3.0...v18.3.2)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
closes actions#377
also I closed the stale process once the stale label is removed since the following process is regarding the closing and it should simply not occur if no longer stale
@C0ZEN C0ZEN requested a review from a team as a code owner April 28, 2021 21:55
Copy link
Contributor Author

@C0ZEN C0ZEN left a comment

Choose a reason for hiding this comment

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

Merge LGTM

Copy link
Collaborator

@luketomlinson luketomlinson left a comment

Choose a reason for hiding this comment

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

LGTM @C0ZEN 👍

@luketomlinson luketomlinson merged commit 440fb17 into actions:main Apr 30, 2021
@C0ZEN C0ZEN deleted the feature/remove-stale-when-updated branch April 30, 2021 13:44
@pllim
Copy link

pllim commented May 19, 2021

Thank you, all! I appreciate GitHub's continued support for OSS folks.

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.

Split remove-stale-when-updated for issues and PRs
3 participants