-
Notifications
You must be signed in to change notification settings - Fork 873
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
Document issue triage labels #3990
base: main
Are you sure you want to change the base?
Conversation
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
This process is still be trialled by the GC so I it's still applicable. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
I believe this is ready for review. Not much has changed but we've been using this process for a while without issue now. It can be extended in the future as we refine the process. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New labels and process LGTM
@dyladan this PR seems to focus on the definition of the roles and tags, it doesn't cover how the state machine would work - e.g. the diagram in this issue description #3821. Do you intend to make this a complete PR by adding the state machine or keeping it small? If the answer is the latter, consider updating the PR title and description. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
issue-management.md
Outdated
- Collaborator: | ||
- Person(s) that are actually doing the work related to the ticket. Once work is done, | ||
they work with the Reviewer to get feedback implemented and complete the work. They | ||
are responsible for making sure issue status labels are up to date. | ||
- Reviewer: | ||
- Person whose Approval is needed to merge the PR. | ||
- Sponsor: | ||
- This person, typically a technical committee member or someone the TC has assigned, is responsible for the completion of the issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's a slightly refined definition in open-telemetry/community#2070
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think it would be better to simply link to that page for all roles instead of redefining them here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trask would you take a look and see if this can be resolved? Thanks!
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
I'll update it today so it can be merged |
@trask Maybe a last look/iteration? |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
Relates to #3821
The GC is currently working on a new process for triaging and prioritizing the specification issue backlog. This document outlines the process and the labels currently being used.