Skip to content

Commit 2c485a6

Browse files
authored
Merge pull request #1527 from mcanouil/docs/triaging-guide
docs: Add issue and discussion triage guide documentation
2 parents 3d838c6 + 27d45f5 commit 2c485a6

File tree

1 file changed

+377
-0
lines changed

1 file changed

+377
-0
lines changed

docs/triaging/index.qmd

Lines changed: 377 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,377 @@
1+
---
2+
title: "Issue and Discussion Triage"
3+
description: |
4+
This guide provides instructions for effectively triaging issues and discussions on Quarto CLI GitHub repository.
5+
The primary goal is to ensure that all issues and discussions are managed efficiently, helping maintain a clear and organized project workflow.
6+
engine: markdown
7+
toc-depth: 4
8+
---
9+
10+
## Introduction
11+
12+
### Purpose
13+
14+
This guide provides instructions for effectively triaging issues and discussions on a Quarto CLI GitHub repository.
15+
The primary goal is to ensure that all issues and discussions are managed efficiently, helping maintain a clear and organized project workflow.
16+
17+
### Scope
18+
19+
The document covers the processes for identifying, reviewing, and managing issues and discussions.
20+
This includes assigning responsibilities, prioritizing tasks, and ensuring all issues are directed towards the appropriate project milestones.
21+
22+
### Tech Lead Engagement
23+
24+
Tech leads and other maintainers will be actively engaged in the ongoing triage of new project issues.
25+
Their involvement is crucial to maintaining the project's organization and ensuring that issues are handled promptly and effectively.
26+
27+
### Organization of Issues
28+
29+
Issues will be continuously organized into milestones based on a clear [Planning Horizon](#planning-horizon).
30+
This ensures that the team has a structured approach to managing the project's progress and addressing issues in a timely manner.
31+
32+
### All Issues and Discussions Managed in Quarto CLI Repository
33+
34+
To simplify the team's workflow, it has been decided that all issues will live in the [`quarto-cli`](https://github.com/quarto-dev/quarto-cli) repository rather than being distributed across multiple repositories.
35+
For example, issues related to the [`quarto-web`](https://github.com/quarto-dev/quarto-web) documentation website will be logged in the [`quarto-cli`](https://github.com/quarto-dev/quarto-cli) issues, even if they are not technically CLI issues.
36+
37+
### Planning Horizon
38+
39+
The team's "Planning Horizon" will be limited to the next Milestone.
40+
Issues will not be triaged to milestones beyond this Planning Horizon.
41+
All issues beyond the next Milestone will be triaged to the "Future" milestone, ensuring a focused and manageable workload for the team.
42+
43+
| Milestone | Meaning | Timeline |
44+
|-----------------------------------------------------------------|------------------------------------------------|------------|
45+
| [Hot-fix](https://github.com/quarto-dev/quarto-cli/milestone/7) | We will fix this right away. | ASAP |
46+
| Current Release | We are working on this actively. | 1-3 Months |
47+
| Next Release | We will work on this soon. | 2-6 Months |
48+
| [Future](https://github.com/quarto-dev/quarto-cli/milestone/2) | We don't know when or if we will work on this. | Unclear |
49+
50+
{{< pagebreak >}}
51+
52+
## Triaging Issues
53+
54+
### Reproducibility
55+
56+
- Follow the steps provided to reproduce the issue.
57+
- Confirm the issue exists and gather any additional information if needed.
58+
59+
### Comments
60+
61+
- Leave informative and respectful comments.
62+
- Ask for additional details if needed and provide clear guidance.
63+
64+
### Identification
65+
66+
#### Labels
67+
68+
- Use clear and consistent labels such as [`bug`](https://github.com/quarto-dev/quarto-cli/labels/bug), [`enhancement`](https://github.com/quarto-dev/quarto-cli/labels/enhancement), [`support`](https://github.com/quarto-dev/quarto-cli/labels/support), [`maintenance`](https://github.com/quarto-dev/quarto-cli/labels/maintenance), and [`documentation`](https://github.com/quarto-dev/quarto-cli/labels/documentation) to define the nature of the issue.
69+
Issues are expected to have exactly one of these labels.
70+
- Assign additional [labels](https://github.com/quarto-dev/quarto-cli/labels/) to categories issues based on their type such as `html`, `website`, `installers`, `crossref`, *etc.*
71+
- Create custom labels if necessary to better categories issues.
72+
73+
#### Prioritization
74+
75+
- Assess the severity and impact of issues to determine priority, *e.g.*:
76+
- use the [`regression` label](https://github.com/quarto-dev/quarto-cli/labels/regression) for bugs that appeared in a recent release
77+
- use the [`Hot-fix` milestone](https://github.com/quarto-dev/quarto-cli/milestone/7) or the [`Future` milestone](https://github.com/quarto-dev/quarto-cli/milestone/2), see [Planning Horizon](#planning-horizon).
78+
- use the number of reactions to the issue to gauge its importance to the community.
79+
- Ensure a milestone is assigned to each issue to indicate its priority and expected completion date.
80+
- Review the [issues not assigned to a milestone](https://github.com/quarto-dev/quarto-cli/issues?q=is%3Aissue%20state%3Aopen%20no%3Amilestone) and assign them accordingly.
81+
82+
### Assignment and Ownership
83+
84+
- Assign issues to appropriate team members based on expertise and workload (add the [`triaged-to` label](https://github.com/quarto-dev/quarto-cli/labels/triaged-to)).
85+
86+
When an issue is assigned, it is given an owner.
87+
The owner is the individual _currently responsible for moving the issue forward_.
88+
89+
**Note**: Ownership does not imply that the person must resolve the issue themselves or be actively working on it at all times.
90+
91+
The [`triaged-to` label](https://github.com/quarto-dev/quarto-cli/labels/triaged-to) is used when assigning ownership to indicate that an issue has been reassigned to someone else.
92+
93+
- **Healthy actions taken by the owner include**:
94+
- **Re-assigning to a more appropriate owner**: This can be done, sometimes after consulting with the team, to ensure the issue is handled by the most suitable person.
95+
- **Triaging to a better Milestone**: Reassessing and assigning the issue to a more relevant Milestone, again, sometimes after team consultation.
96+
- **Attempting to reproduce the issue**: Providing more information by trying to replicate the issue.
97+
- **Resolving the issue**:
98+
- [*Close as completed*]{.badge .rounded-pill .text-bg-primary}: The issue has been resolved with a pull request linked to the issue.
99+
- [*Close as not planned*]{.badge .rounded-pill .text-bg-primary}: The issue is not planned to be resolved, was marked by CI/CD as [`stale`](https://github.com/quarto-dev/quarto-cli/labels/stale) because of a lack of reproducibility ([`needs-repro` label](https://github.com/quarto-dev/quarto-cli/labels/needs-repro)), is actually a [`support`](https://github.com/quarto-dev/quarto-cli/labels/support) / discussion issue, or is no longer relevant.
100+
- [*Close as duplicate*]{.badge .rounded-pill .text-bg-primary}: The issue is a duplicate of another issue ([`duplicate`](https://github.com/quarto-dev/quarto-cli/labels/duplicate)).
101+
- **Anti-patterns and discouraged actions**:
102+
- **Un-assigning the owner**: Making the issue ownerless is discouraged as we promote a workflow where every issue has a designated owner.
103+
The exception to this rule is for issues in the Future milestone, where having assigned owners is less critical.
104+
- **Un-assigning the milestone**: Removing the milestone from an issue without reassigning it effectively un-triages it.
105+
We promote continuous refinement of issue milestones.
106+
- **Closing without information**: Closing issues without providing a clear explanation or context is discouraged.
107+
Clear communication is essential for maintaining transparency and understanding.
108+
109+
An owner might encounter obstacles that prevent them from moving an issue forward.
110+
They might need additional information or need to re-balance their workload.
111+
In such cases, we encourage the following actions:
112+
113+
- **Flag the issue with [`needs-discussion`](https://github.com/quarto-dev/quarto-cli/labels/needs-discussion)**: Mark the issue with this label so it can be reviewed at the next team sync.
114+
- **Seek help via Slack**: Post a question in the team Slack channel to get immediate assistance or input from team members.
115+
116+
{{< pagebreak >}}
117+
118+
## Triaging Discussions
119+
120+
### Identification
121+
122+
#### Categories
123+
124+
- Ensure discussions are categorized based on their nature, such as [`Features Requests`](https://github.com/quarto-dev/quarto-cli/discussions/categories/feature-requests), [`Q&A`](https://github.com/quarto-dev/quarto-cli/discussions/categories/q-a), and [`Show and tell`](https://github.com/quarto-dev/quarto-cli/discussions/categories/show-and-tell).
125+
126+
#### Labels
127+
128+
- Assign [labels](https://github.com/quarto-dev/quarto-cli/labels/) to categorize issues based on their type such as `html`, `website`, `installers`, `crossref`, *etc.*
129+
130+
### Participation
131+
132+
- Engage in discussions with clear, respectful, and constructive input.
133+
- Encourage community involvement and acknowledge valuable contributions.
134+
135+
### Reproducibility
136+
137+
- Follow the steps provided to reproduce the user's use case.
138+
- Gather any additional information if needed.
139+
140+
### Comments
141+
142+
- Leave informative and respectful comments.
143+
- Ask for additional details if needed and provide clear guidance (see [Reply Templates](#reply-templates)).
144+
145+
### Upgrading to Issues
146+
147+
- If a discussion appears to be a bug report and can be reproduced, use [*Create issue from discussion*]{.badge .rounded-pill .text-bg-primary} GitHub feature to convert it into an issue.
148+
- [*Close as outdated*]{.badge .rounded-pill .text-bg-primary} the discussion and link to the newly created issue.
149+
150+
### Resolution
151+
152+
- For [`Q&A` discussions](https://github.com/quarto-dev/quarto-cli/discussions/categories/q-a), mark the appropriate answer as the solution if not done already.
153+
- Do not close discussions unless it has been upgraded to an issue.
154+
155+
{{< pagebreak >}}
156+
157+
## Best Practices
158+
159+
### Communication
160+
161+
- Maintain clear and respectful communication at all times.
162+
- Provide concise and constructive feedback.
163+
164+
### Consistency
165+
166+
- Ensure labels, milestones, and comments are used consistently across the repository.
167+
- Regularly review and update triaging practices to maintain effectiveness.
168+
169+
{{< pagebreak >}}
170+
171+
## Appendix
172+
173+
### Templates
174+
175+
#### Issue Reporting Template
176+
177+
- [Bug report](https://github.com/quarto-dev/quarto-cli/tree/main/.github/ISSUE_TEMPLATE/bug_report.yml)
178+
- [Documentation issue](https://github.com/quarto-dev/quarto-cli/blob/main/.github/ISSUE_TEMPLATE/documentation.yml)
179+
180+
#### Discussion Template
181+
182+
- [Feature Requests](https://github.com/quarto-dev/quarto-cli/blob/main/.github/DISCUSSION_TEMPLATE/feature-requests.yml)
183+
- [Q&A](https://github.com/quarto-dev/quarto-cli/blob/main/.github/DISCUSSION_TEMPLATE/q-a.yml)
184+
- [Show and Tell](https://github.com/quarto-dev/quarto-cli/blob/main/.github/DISCUSSION_TEMPLATE/show-and-tell.yml)
185+
186+
### Reply Templates
187+
188+
#### Code of Conduct
189+
190+
````{.markdown shortcodes=false}
191+
> [!IMPORTANT]
192+
> We appreciate your interest in the Quarto project, but the content above violates our [Code of Conduct](https://github.com/quarto-dev/quarto-cli?tab=coc-ov-file#readme).
193+
>
194+
> As a member of the Quarto community, you are expected to follow our Code of Conduct, which outlines the standards of behavior and communication that we value. The Code of Conduct applies to all online interactions within the Quarto project, and it is enforced by the Quarto team. By participating in the Quarto community, you agree to abide by the Code of Conduct and respect the rights and dignity of others. You can find the Code of Conduct at <https://github.com/quarto-dev/quarto-cli?tab=coc-ov-file#readme>.
195+
>
196+
> Thank you for your cooperation and contribution. 🙏
197+
````
198+
199+
#### Ask reformat
200+
201+
````{.markdown shortcodes=false}
202+
Could you properly format your post using code blocks for code and terminal outputs? Thanks.
203+
If your code contains code blocks, you need to enclose it using more backticks, *i.e.*, usually four ` ```` `.
204+
See <https://quarto.org/bug-reports.html#formatting-make-githubs-markdown-work-for-us>.
205+
````
206+
207+
#### Ask reproducible example
208+
209+
``````{.markdown shortcodes=false}
210+
Could you share a **small self-contained "working" (reproducible)** example to work with, _i.e._, a complete Quarto document or a Git repository? The goal is to make it as easy as possible for us to recreate your problem so that we can fix it: please help us help you! Thanks.
211+
212+
---
213+
214+
You can share a **self-contained "working" (reproducible)** Quarto document using the following syntax, _i.e._, using more backticks than you have in your document (usually four ` ```` `).
215+
See <https://quarto.org/bug-reports.html#small-is-beautiful-aim-for-a-single-document-with-10-lines>.
216+
217+
If you have multiple files (and if it is **absolutely required** to have multiple files), please share as a Git repository.
218+
219+
<table>
220+
<tr><th>R</th><th>Python</th></tr>
221+
<tr><td>
222+
223+
`````md
224+
````qmd
225+
---
226+
title: "Reproducible Quarto Document"
227+
format: html
228+
engine: knitr
229+
---
230+
231+
This is a reproducible Quarto document.
232+
233+
{{< lipsum 1 >}}
234+
235+
```{r}
236+
x <- c(1, 2, 3, 4, 5)
237+
y <- c(1, 4, 9, 16, 25)
238+
239+
plot(x, y)
240+
```
241+
242+
![An image]({{< placeholder 600 400 >}}){#fig-placeholder}
243+
244+
{{< lipsum 1 >}}
245+
246+
The end after @fig-placeholder.
247+
````
248+
`````
249+
250+
</td><td>
251+
252+
`````md
253+
````qmd
254+
---
255+
title: "Reproducible Quarto Document"
256+
format: html
257+
engine: jupyter
258+
---
259+
260+
This is a reproducible Quarto document.
261+
262+
{{< lipsum 1 >}}
263+
264+
```{python}
265+
import matplotlib.pyplot as plt
266+
267+
x = [1, 2, 3, 4, 5]
268+
y = [1, 4, 9, 16, 25]
269+
270+
plt.plot(x, y)
271+
plt.show()
272+
```
273+
274+
![An image]({{< placeholder 600 400 >}}){#fig-placeholder}
275+
276+
{{< lipsum 1 >}}
277+
278+
The end after @fig-placeholder.
279+
````
280+
`````
281+
282+
</td></tr>
283+
</table>
284+
285+
Additionally and if not already given, please share the output of `quarto check` within a code block (*i.e.*, using three backticks ` ```txt `), see <https://quarto.org/bug-reports.html#check>.
286+
``````
287+
288+
#### Reproducible example
289+
290+
``````{.markdown shortcodes=false}
291+
---
292+
293+
You can share a **self-contained "working" (reproducible)** Quarto document using the following syntax, _i.e._, using more backticks than you have in your document (usually four ` ```` `).
294+
See <https://quarto.org/bug-reports.html#small-is-beautiful-aim-for-a-single-document-with-10-lines>.
295+
296+
If you have multiple files (and if it is **absolutely required** to have multiple files), please share as a Git repository.
297+
298+
<table>
299+
<tr><th>R</th><th>Python</th></tr>
300+
<tr><td>
301+
302+
`````md
303+
````qmd
304+
---
305+
title: "Reproducible Quarto Document"
306+
format: html
307+
engine: knitr
308+
---
309+
310+
This is a reproducible Quarto document.
311+
312+
{{< lipsum 1 >}}
313+
314+
```{r}
315+
x <- c(1, 2, 3, 4, 5)
316+
y <- c(1, 4, 9, 16, 25)
317+
318+
plot(x, y)
319+
```
320+
321+
![An image]({{< placeholder 600 400 >}}){#fig-placeholder}
322+
323+
{{< lipsum 1 >}}
324+
325+
The end after @fig-placeholder.
326+
````
327+
`````
328+
329+
</td><td>
330+
331+
`````md
332+
````qmd
333+
---
334+
title: "Reproducible Quarto Document"
335+
format: html
336+
engine: jupyter
337+
---
338+
339+
This is a reproducible Quarto document.
340+
341+
{{< lipsum 1 >}}
342+
343+
```{python}
344+
import matplotlib.pyplot as plt
345+
346+
x = [1, 2, 3, 4, 5]
347+
y = [1, 4, 9, 16, 25]
348+
349+
plt.plot(x, y)
350+
plt.show()
351+
```
352+
353+
![An image]({{< placeholder 600 400 >}}){#fig-placeholder}
354+
355+
{{< lipsum 1 >}}
356+
357+
The end after @fig-placeholder.
358+
````
359+
`````
360+
361+
</td></tr>
362+
</table>
363+
364+
Additionally and if not already given, please share the output of `quarto check` within a code block (*i.e.*, using three backticks ` ```txt `), see <https://quarto.org/bug-reports.html#check>.
365+
``````
366+
367+
#### Accessibility: descriptive link text
368+
369+
````{.markdown shortcodes=false}
370+
Accessibility: For better accessibility, please ensure link text is descriptive and meaningful rather than using generic terms such as "here". This helps all users understand the link's destination. See [Writing meaningful link text (WCAG)](https://www.wcag.com/blog/writing-meaningful-link-text/).
371+
````
372+
373+
#### Accessibility: alternative text
374+
375+
````{.markdown shortcodes=false}
376+
Accessibility: To improve accessibility, please add alternative text to your screenshots. This helps all users, including those using screen readers, to understand the context of the images. A brief description can make a big difference! See [Good Alt Text, Bad Alt Text — Making Your Content Perceivable](https://www.wcag.com/blog/good-alt-text-bad-alt-text-making-your-content-perceivable/).
377+
````

0 commit comments

Comments
 (0)