Skip to content

chore: add stale bot #4048

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

Merged
merged 1 commit into from
Aug 26, 2021
Merged

chore: add stale bot #4048

merged 1 commit into from
Aug 26, 2021

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Aug 26, 2021

@jawnsy brought this up the other day. Then GitHub mentioned we should it so I thought why not πŸ€·β€β™‚οΈ

I will take responsibility if it goes horribly wrong πŸ˜‚

The way this works is we have a stale.yml under .github which is the configuration and then @jawnsy enabled Stale bot it at the cdr org level.

Fixes N/A

@jsjoeio jsjoeio added the chore Related to maintenance or clean up label Aug 26, 2021
@jsjoeio jsjoeio added this to the 3.12.0 milestone Aug 26, 2021
@jsjoeio jsjoeio self-assigned this Aug 26, 2021
@jsjoeio jsjoeio requested a review from a team as a code owner August 26, 2021 23:10
@github-actions
Copy link

github-actions bot commented Aug 26, 2021

✨ Coder.com for PR #4048 deployed! It will be updated on every commit.

@jsjoeio jsjoeio force-pushed the jsjoeio-add-stale branch from cfbb667 to 6c6230e Compare August 26, 2021 23:20
name: Mark stale issues and pull requests

on:
schedule:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's better to use the stale config and have the external Stale app take care of this. A common config across our repos also makes it easier for someone to switch between our repos

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good call!

@codecov
Copy link

codecov bot commented Aug 26, 2021

Codecov Report

Merging #4048 (eed4e64) into main (6c6446f) will not change coverage.
The diff coverage is n/a.

❗ Current head eed4e64 differs from pull request most recent head dbd1a22. Consider uploading reports for the commit dbd1a22 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4048   +/-   ##
=======================================
  Coverage   63.51%   63.51%           
=======================================
  Files          36       36           
  Lines        1872     1872           
  Branches      379      379           
=======================================
  Hits         1189     1189           
  Misses        580      580           
  Partials      103      103           

Continue to review full report at Codecov.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update 6c6446f...dbd1a22. Read the comment docs.

@jsjoeio jsjoeio force-pushed the jsjoeio-add-stale branch from 6c6230e to da2da4b Compare August 26, 2021 23:27
@jsjoeio jsjoeio force-pushed the jsjoeio-add-stale branch from da2da4b to dbd1a22 Compare August 26, 2021 23:30
@jsjoeio jsjoeio enabled auto-merge August 26, 2021 23:32
@jsjoeio jsjoeio merged commit b5afb05 into main Aug 26, 2021
@jsjoeio jsjoeio deleted the jsjoeio-add-stale branch August 26, 2021 23:48
@jsjoeio jsjoeio removed this from the 3.12.0 milestone Aug 30, 2021
@jsjoeio jsjoeio mentioned this pull request Aug 30, 2021
@Lemmingh
Copy link

I guess you forgot to set exemptLabels and exemptMilestones. An On Deck bug is closed:
#1105

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Sep 24, 2021

@Lemmingh thanks for pointing that out!

I guess we should add that. If no one responds though, it's still probably stale and makes sense to close, but I'll open an issue to do this.

@bpmct
Copy link
Member

bpmct commented Sep 27, 2021

Does that mean #33 will be considered stale at some point?

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Sep 27, 2021

@bpmct yes if there is no activity for 180 days!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Related to maintenance or clean up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants