Skip to content
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

chore: regularly close stale issues #2533

Merged
merged 2 commits into from
Nov 10, 2021

Conversation

rauno56
Copy link
Member

@rauno56 rauno56 commented Oct 13, 2021

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
Copy link

codecov bot commented Oct 13, 2021

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
Copy link
Member

dyladan commented Oct 27, 2021

Some reason this is a draft?

edit: should have read the description :)

@dyladan
Copy link
Member

dyladan commented Nov 10, 2021

@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
Copy link
Member Author

rauno56 commented Nov 10, 2021

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

@dyladan
Copy link
Member

dyladan commented Nov 10, 2021

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