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

Add schedule for staling issues #4752

Merged
merged 5 commits into from
Sep 27, 2022

Conversation

sunjayBhatia
Copy link
Member

@sunjayBhatia sunjayBhatia commented Sep 26, 2022

To whittle down issues a little, to ensure we're focusing on things that are deemed important and are getting proper community attention.

Stale after 60 days, closed after another 30

Exempted a few labels from being staled as well.

Stale after 60 days, closed after 90

Exempted a few labels from being staled as well.

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
@sunjayBhatia sunjayBhatia requested a review from a team as a code owner September 26, 2022 15:15
@sunjayBhatia sunjayBhatia requested review from tsaarni and skriss and removed request for a team September 26, 2022 15:15
@sunjayBhatia sunjayBhatia added the release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes. label Sep 26, 2022
.github/workflows/stale.yaml Outdated Show resolved Hide resolved
.github/workflows/stale.yaml Outdated Show resolved Hide resolved
priority/important-soon added as an exempt label

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
@codecov
Copy link

codecov bot commented Sep 26, 2022

Codecov Report

Merging #4752 (91e3fce) into main (6bde6ff) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 91e3fce differs from pull request most recent head bc9189e. Consider uploading reports for the commit bc9189e to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4752      +/-   ##
==========================================
- Coverage   75.67%   75.66%   -0.01%     
==========================================
  Files         140      140              
  Lines       16836    16836              
==========================================
- Hits        12740    12739       -1     
- Misses       3842     3843       +1     
  Partials      254      254              
Impacted Files Coverage Δ
internal/sorter/sorter.go 97.32% <0.00%> (-0.54%) ⬇️

@sunjayBhatia
Copy link
Member Author

modified the project board to move existing priority/important-soon items to priority/important-longterm, planned things on the project board now have priority/important-soon

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
duplicate of stale pr/issue message

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
Copy link
Member

@skriss skriss left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for updating the messages.

I'm in favor of doing this, along with better triage of issues as they're opened, to better manage our backlog and reflect to our users what our priorities are. Having a smaller/more focused backlog will be more useful for both maintainers and users, while not preventing anyone from "helping themselves" if they have a specific issue they want to contribute on.

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
@sunjayBhatia sunjayBhatia merged commit 5af0a42 into projectcontour:main Sep 27, 2022
@sunjayBhatia sunjayBhatia deleted the stale-issue-policy branch September 27, 2022 15:55
izturn pushed a commit to izturn/contour that referenced this pull request Sep 29, 2022
Stale after 60 days, closed after another 30

Exempted a few labels from being staled as well.

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
Signed-off-by: Gang Liu <gang.liu@daocloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants