-
Notifications
You must be signed in to change notification settings - Fork 105
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
[close #528] add workflows to check staleness and PR title #529
Conversation
Signed-off-by: Jian Zhang <zjsariel@gmail.com>
Signed-off-by: Jian Zhang <zjsariel@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
days-before-stale: 30 | ||
stale-issue-message: 'This issue is stale because it has been open 30 days with no activity. Remove the status/stale label or comment or this will be closed in 5 days.' | ||
stale-issue-label: 'status/stale' | ||
days-before-issue-close: -1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be 5?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
currently we don't need to close stale issues since almost all these issues need to be solved.
days-before-stale: 30 | ||
stale-issue-message: 'This issue is stale because it has been open 30 days with no activity. Remove the status/stale label or comment or this will be closed in 5 days.' | ||
stale-issue-label: 'status/stale' | ||
days-before-issue-close: -1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
currently we don't need to close stale issues since almost all these issues need to be solved.
.github/workflows/stale-checker.yml
Outdated
- uses: actions/stale@v4 | ||
with: | ||
days-before-stale: 30 | ||
stale-issue-message: 'This issue is stale because it has been open 30 days with no activity. Remove the status/stale label or comment or this will be closed in 5 days.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
stale-issue-message: 'This issue is stale because it has been open 30 days with no activity. Remove the status/stale label or comment or this will be closed in 5 days.' | |
stale-issue-message: 'This issue is stale because it has been open 30 days with no activity.' |
Signed-off-by: Jian Zhang <zjsariel@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #529 +/- ##
============================================
+ Coverage 30.88% 30.94% +0.06%
- Complexity 1270 1274 +4
============================================
Files 278 278
Lines 17343 17343
Branches 1975 1975
============================================
+ Hits 5356 5367 +11
+ Misses 11393 11381 -12
- Partials 594 595 +1
Continue to review full report at Codecov.
|
/merge |
/run-all-tests |
What problem does this PR solve?
Issue Number: [close #528]
Problem Description: as the issue said:
What is changed and how it works?
add github actions
Code changes
Check List for Tests
This PR has been tested by the at least one of the following methods:
Side effects
Related changes