Skip to content

chore: regularly close stale issues - #2533

Merged
dyladan merged 2 commits into
open-telemetry:mainfrom
rauno56:chore/add-stale-issue-gha
Nov 10, 2021
Merged

chore: regularly close stale issues#2533
dyladan merged 2 commits into
open-telemetry:mainfrom
rauno56:chore/add-stale-issue-gha

Conversation

@rauno56

@rauno56 rauno56 commented Oct 13, 2021

Copy link
Copy Markdown
Member

Which problem is this PR solving?

We have an increasing number of unattended issues. Some of them are still relevant, many aren't. Setting up automation will ping the authors and participants to make sure there's either advancement on the issues/PRs or those will be cleaned up.

Adding never-stale label to either an issue or a PR will disable that feature.

The job is run weekly. The first run will be painful and will mark many issues stale once.

There's a companion PR for contrib: open-telemetry/opentelemetry-js-contrib#697
I'd leave this one as a draft for now and see how it goes in the first week or two in contrib, not to overwhelm ourselves.

@rauno56 rauno56 changed the title feat: add GHA to mark issues and PRs stale and close them chore: regularly close stale issues Oct 13, 2021
@codecov

codecov Bot commented Oct 13, 2021

Copy link
Copy Markdown

Codecov Report

Merging #2533 (34588a4) into main (2e378fd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2533   +/-   ##
=======================================
  Coverage   93.04%   93.04%           
=======================================
  Files         138      138           
  Lines        5093     5093           
  Branches     1096     1096           
=======================================
  Hits         4739     4739           
  Misses        354      354           

@dyladan

dyladan commented Oct 27, 2021

Copy link
Copy Markdown
Member

Some reason this is a draft?

edit: should have read the description :)

@dyladan

dyladan commented Nov 10, 2021

Copy link
Copy Markdown
Member

@rauno56 some reason this is a draft? I think i'd like to merge this if you're ready

@rauno56
rauno56 marked this pull request as ready for review November 10, 2021 17:30
@rauno56
rauno56 requested a review from a team November 10, 2021 17:30
@rauno56

rauno56 commented Nov 10, 2021

Copy link
Copy Markdown
Member Author

Wanted to give the setup in the other repo a few cycles before merging this, just in case...

@dyladan

dyladan commented Nov 10, 2021

Copy link
Copy Markdown
Member

Nobody on the SIG call had any complaints about the contrib action so it should be good here.

@dyladan
dyladan merged commit e8e787d into open-telemetry:main Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants