chore(ci): add stale issue triage workflow#497
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #497 +/- ##
==========================================
+ Coverage 80.93% 80.96% +0.02%
==========================================
Files 24 24
Lines 5482 5489 +7
Branches 247 247
==========================================
+ Hits 4437 4444 +7
Misses 968 968
Partials 77 77
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| days-before-issue-close: 30 | ||
| stale-issue-label: 'stale' | ||
| stale-issue-message: > | ||
| This issue has been inactive for 180 days and labeled **stale** Comment to keep it open. It will be closed in 30 days if there is no activity. |
There was a problem hiding this comment.
| This issue has been inactive for 180 days and labeled **stale** Comment to keep it open. It will be closed in 30 days if there is no activity. | |
| This issue has been inactive for 180 days and labeled **stale**, comment to keep it open. It will be closed in 30 days if there is no activity. |
There was a problem hiding this comment.
Updated in the latest rebase, thanks.
Dmenec
left a comment
There was a problem hiding this comment.
Concept ACK. Just a little nit on close reason.
| This issue has been inactive for 180 days and labeled **stale** Comment to keep it open. It will be closed in 30 days if there is no activity. | ||
| close-issue-message: > | ||
| Closed after 210 days of inactivity. Feel free to reopen if this is still relevant. | ||
| close-issue-reason: completed |
There was a problem hiding this comment.
not_planned fits an inactivity close better (as mentioned in actions/stale#789), and since it's the default option we could just drop the line.
ValuedMammal
left a comment
There was a problem hiding this comment.
I would be generous and leave PRs open for 180 days. Suggest removing discussion from exempt labels. If there are long running discussions, they can happen in https://github.com/bitcoindevkit/bdk_wallet/discussions
0e0f788 to
bb1fc1f
Compare
@ValuedMammal Thanks, updated the exempt labels. I also adjusted the PR timeout to 180 days. |
Can we update the |
3f18adc to
d466b08
Compare
|
Updated the workflow based on the Lib Standup discussion:
|
notmandatory
left a comment
There was a problem hiding this comment.
ACK d466b08
Great first step in helping close old issues and PRs in this repo. If all goes well I'd like to see this workflow in other repos in the org.
oleonardolima
left a comment
There was a problem hiding this comment.
cACK
I left a few suggestions.
| stale: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/stale@eb5cf3af3ac0a1aa4c9c45633dd1ae542a27a899 # v10 |
There was a problem hiding this comment.
I didn't know that stale was an official action, it can have the version used directly instead of the hash.
There was a problem hiding this comment.
Even though it's an official plugin I think Zizmor action will still complain so I don't see a problem leaving the hash since it returns the same version and is marginally safer.
There was a problem hiding this comment.
Yes, though we allow those in
Line 9 in 58fe631
There was a problem hiding this comment.
Oh didn't know that, I'm fine with it either way.
4d6071a to
1d03c9c
Compare
| ascending: true | ||
| operations-per-run: 600 | ||
| debug-only: ${{ github.event_name == 'pull_request' || github.event.inputs.debug_only == 'true' }} | ||
| enable-statistics: true No newline at end of file |
There was a problem hiding this comment.
nit: just add a newline at EOF here.
1d03c9c to
a63c7a6
Compare
notmandatory
left a comment
There was a problem hiding this comment.
re-ACK a63c7a6
Best to merge this after Monday so we have time to test it with the dry-run button before it does its automatic Monday run.
Description
Part of #410
Closes #386
Adds
.github/workflows/stale.ymlusingactions/stale@v10to automatically label and close inactive issues and abandoned PRs.release,security,audit,do-not-close,good first issuefor issues andblocked,security,do-not-closefor PRNotes to the reviewers
Changelog notice
ci: Automated stale issue and PR triage workflow
Before submitting