Skip to content

Commit df1c26e

Browse files
Add guidance for maintainers about GitHub issue reactions (#2778)
* Add guidance for maintainers about GitHub issue reactions * Fix spelling * Adress review comments * Use American spelling * Update recommended message * Fix typo * Add changes after trialing in repos * Change title
1 parent 574624a commit df1c26e

File tree

5 files changed

+122
-5
lines changed

5 files changed

+122
-5
lines changed

guides/README.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,9 @@ everyone is welcome to contribute to them!
88

99
- [Contributor Guide](./contributor/README.md)
1010
- [Policy on Generative AI Contributions](./contributor/genai.md)
11-
- [Donations of pre-existing code](./contributor/donations.md)
12-
- [Becoming a member of the OpenTelemetry project](./contributor/membership.md)
11+
- [Donations of Pre-Existing Code](./contributor/donations.md)
12+
- [Becoming a Member of the OpenTelemetry Project](./contributor/membership.md)
1313
- [The Contribution Lifecycle](./contributor/processes.md)
1414
- [Maintainer Guide](./maintainer/README.md)
15-
- [Resolving technical conflicts](./maintainer/conflict-resolution.md)
15+
- [Resolving Technical Conflicts](./maintainer/conflict-resolution.md)
16+
- [Managing Popular GitHub Issues](./maintainer/popular-issues.md)

guides/maintainer/README.md

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,23 @@ project.
66

77
## Table of Contents
88

9-
TODO
9+
- [What is a Maintainer?](#what-is-a-maintainer)
10+
- [Resolving Technical Conflicts](#resolving-technical-conflicts)
11+
- [Managing Popular GitHub Issues](#managing-popular-github-issues)
1012

1113
## What is a Maintainer?
1214

1315
See [Maintainer](../contributor/membership.md) for more information.
16+
17+
## Resolving Technical Conflicts
18+
19+
The [Resolving Technical Conflicts](conflict-resolution.md) guide contains
20+
advice on handling scenarios where consensus cannot be reached within a SIG,
21+
and an escalation path may be sought.
22+
23+
## Managing Popular GitHub Issues
24+
25+
The [Managing Popular GitHub Issues](popular-issues.md) doc
26+
contains guidance for maintainers to encourage user feedback via GitHub
27+
reactions, and how to balance those with other work to prioritize work for their
28+
SIG.

guides/maintainer/conflict-resolution.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# Resolving technical conflicts within a SIG
1+
# Resolving Technical Conflicts Within a SIG
22

33
From time to time, the Maintainers for a given OpenTelemetry SIG may be unable
44
to reach consensus on a technical issue. While it's healthy and appropriate to
11 KB
Loading
Lines changed: 101 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,101 @@
1+
# Managing Popular GitHub Issues
2+
3+
Within OpenTelemetry, we encourage end-users and community members to leave 👍
4+
reactions on GitHub issues they'd like to be worked on, as indicated in our
5+
[website](https://opentelemetry.io/community). This can help maintainers to:
6+
7+
- Get a high-level signal that measures the interest from the community in
8+
particular issues, in a way that can be evaluated and compared across one or
9+
many repositories.
10+
- Avoid redundant, unproductive comments like _"I have this issue too"_, which
11+
are difficult to compare across issues, and provide no context for triage.
12+
13+
In addition to these benefits, this standardized approach helps the wider
14+
OpenTelemetry community by making it easier for end-users to highlight important
15+
issues without requiring them to join SIG meetings or Slack channels. This also
16+
allows the End-User SIG to focus on specific topics that demonstrate how user
17+
input shapes OpenTelemetry, boosting engagement and strengthening the feedback
18+
loop.
19+
20+
## Popularity Does Not Imply Priority
21+
22+
OpenTelemetry SIGs have the autonomy to organize their work and manage roadmaps
23+
independently, balancing their priorities while remaining aligned with the
24+
general direction of the project.
25+
26+
While maintainers may use these reactions to prioritize issues, they’re **not
27+
expected to solely base their issue prioritization on popularity**. Cases where
28+
popular issues are not prioritized include (but are not limited to):
29+
30+
* High-priority SIG work (e.g. security, spec compliance).
31+
* Issues that require other work to be completed before they can be addressed.
32+
* Requests or proposals not aligning with the direction of the project.
33+
34+
## Communicating Priorities
35+
36+
Triagers should keep the most voted issues up-to-date on a best-effort basis,
37+
and communicate SIG priorities back to users if other work is taking precedence
38+
before a given issue is prioritized. This avoids frustration for end-users and
39+
gives them a way to see progress towards the issues they find the most
40+
important.
41+
42+
How this communication is carried out within the most voted issues, or how many
43+
issues are considered, is decided by SIG maintainers and triagers. This may
44+
involve pointing users to a project board where issues are prioritized, meeting
45+
notes, or simply updating popular issues regularly.
46+
47+
## Finding the Most Popular Issues
48+
49+
The most popular issues for a given SIG can be found:
50+
51+
- For a particular repo: see
52+
[this sample filter](https://github.com/open-telemetry/community/issues?q=is%3Aissue+is%3Aopen+sort%3Areactions-%2B1-desc)
53+
for the `community` repo.
54+
- Across multiple repos: see
55+
[this sample filter](https://github.com/search?q=is%3Aissue+is%3Aopen+sort%3Areactions-%2B1-desc+repo%3Aopen-telemetry%2Fopentelemetry.io+repo%3Aopen-telemetry%2Fcommunity+&type=issues)
56+
for the `community` and `opentelemetry.io` repos.
57+
58+
## Recommended Footnote on Issue Templates
59+
60+
We recommend using the following footnote on issue templates to ensure users
61+
have direct access to guidance on how OpenTelemetry uses issue reactions.
62+
63+
### Markdown Templates
64+
65+
Add this to the bottom of the `.md` file.
66+
67+
```markdown
68+
<sub>**Tip**: [React](https://github.blog/news-insights/product-news/add-reactions-to-pull-requests-issues-and-comments/) with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it. Learn more [here](https://opentelemetry.io/community/end-user/issue-participation/).</sub>
69+
```
70+
71+
Which renders as:
72+
73+
<sub>**Tip**: [React](https://github.blog/news-insights/product-news/add-reactions-to-pull-requests-issues-and-comments/) with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it. Learn more [here](https://opentelemetry.io/community/end-user/issue-participation/).</sub>
74+
75+
### Issue Forms
76+
77+
As covered in this
78+
[feedback discussion topic](https://github.com/orgs/community/discussions/63402#discussioncomment-10341167),
79+
it is not possible to create `markdown` elements that render on the resulting
80+
issue body. As such, we recommend using a dropdown element with a default
81+
option.
82+
83+
```yaml
84+
- type: dropdown
85+
attributes:
86+
label: Tip
87+
description: This element is static, used to render a helpful sub-heading for end-users and community members to help prioritize issues. Please leave as is.
88+
options:
89+
- <sub>[React](https://github.blog/news-insights/product-news/add-reactions-to-pull-requests-issues-and-comments/) with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it. Learn more [here](https://opentelemetry.io/community/end-user/issue-participation/).</sub>
90+
default: 0
91+
```
92+
93+
In the issue form, this renders as:
94+
95+
![issue-form-tip.png](issue-form-tip.png)
96+
97+
In the resulting issue, this renders as:
98+
99+
### Tip
100+
101+
<sub>[React](https://github.blog/news-insights/product-news/add-reactions-to-pull-requests-issues-and-comments/)with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it. Learn more [here](https://opentelemetry.io/community/end-user/issue-participation/).</sub>

0 commit comments

Comments
 (0)