Skip to content

chore: update stale action#15665

Merged
Ovilia merged 3 commits into
masterfrom
fix-stale
Sep 3, 2021
Merged

chore: update stale action#15665
Ovilia merged 3 commits into
masterfrom
fix-stale

Conversation

@Ovilia
Copy link
Copy Markdown
Contributor

@Ovilia Ovilia commented Sep 3, 2021

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Apache ECharts PMC agrees on that stale issues (which haven't been active for more than 2 years) are most like outdated. In order for developers to focus on more recent problems and feature requests, we deciced to automatically close stale issues with the help of bots. If the problem remain after 2 years, it is recommended to open a new one to get the attention from the community.

Previously, we used https://github.com/probot/stale to mark issues to be stale that have not been active in 2 years and automatically close them after 7 days. But somehow it hasn't been working since Apache ECharts graduated and we failed to understand why, even with the help of ASF infra reinstalling the bot.

So in this PR, we propose to use GitHub stale action to do the same job. It marks issues/PRs that haven't been active for at least 730 days to be stale and close them if no further response happens within 7 days.

@echarts-bot
Copy link
Copy Markdown

echarts-bot Bot commented Sep 3, 2021

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

The pull request is marked to be PR: author is committer because you are a committer of this project.

@Ovilia Ovilia merged commit dc275b5 into master Sep 3, 2021
@echarts-bot
Copy link
Copy Markdown

echarts-bot Bot commented Sep 3, 2021

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants