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] diagram defining different states for issues #36975

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

VihasMakwana
Copy link
Contributor

Description

See #36677

Closes #36677

issue-triaging.md Outdated Show resolved Hide resolved
@VihasMakwana VihasMakwana requested a review from atoulme December 31, 2024 13:42
issue-triaging.md Outdated Show resolved Hide resolved
issue-triaging.md Outdated Show resolved Hide resolved
@mx-psi mx-psi self-requested a review January 7, 2025 09:08
issue-triaging.md Outdated Show resolved Hide resolved
@fatsheep9146
Copy link
Contributor

image

Add a screenshot for current version in case other approvers have interest in this.

@fatsheep9146
Copy link
Contributor

ping @open-telemetry/collector-contrib-approvers for review

n6 -- Yes --> n7["Valid issue and to be fixed <br>by a codeowner"]
n6 -- No --> n8
n7 --> n9
n8 --> n9
Copy link
Member

Choose a reason for hiding this comment

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

There should be some step between help-wanted and closed

n2 -- Yes --> n3
n2 -- No/Need more details --> n4
n2 -- Invalid configuration/alternative available --> n9
n3 -- Invalid Issue --> n9
Copy link
Member

Choose a reason for hiding this comment

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

may be worth giving some examples on invalid issues


```mermaid
flowchart TD
subgraph graph1["**needs-triage**"]
Copy link
Member

Choose a reason for hiding this comment

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

I don't feel like the subgraph adds a lot to it, I think probably removing it would make the diagram a bit less intimidating. What do you think? Is there anything we would lose without it?

n2 -- No/Need more details --> n4
n2 -- Invalid configuration/alternative available --> n9
n3 -- Invalid Issue --> n9
n3 -- Valid Issue --> n6["Codeowner has time<br>to fix it?"]
Copy link
Member

Choose a reason for hiding this comment

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

I feel like the criteria here would be something more like "issue is suitable for non-codeowners"? My expectation is that for bugs at least, if someone signs up as a code owner is because they have time to fix the bug

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

Successfully merging this pull request may close these issues.

Define states an issue can be at
5 participants