-
Notifications
You must be signed in to change notification settings - Fork 65
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
Remove 'new-task' GitHub template #592
Conversation
Too many options here was a bit overwhelming for me. I am not sure what a specific 'Task' label gives us over just blank issues.
@choldgraf I'm trying to go through and look at processes where I'm overwhelmed with the amount of choices, and trying to change to fit. I'm also not sure if 'Goal' is helpful anymore, so we should consider removing that too. |
I like the idea of having a template that nudges me to describe the problem and think about a possible pathway forward, but I do also find the number of choices overwhelming. I almost always choose "New Task" though because it feels low down on the list of importance and, therefore, less disruptive if I generate a few over the course of a single working day (I view "Tasks" like "To-Dos"). The exception to my thinking here is the "New Hub" template, which is pretty well scoped I feel. |
@sgibson91 ya I find the new hub and incident report templates helpful. I use 'blank issue' similar to how you use task. May I suggest we remove everything except new hub and incident report? |
I'd quite like some kind of template for these general things just to avoid blank issue bodies and provide that scoping nudge. What do you think of the Turing Way's "General" issue template? https://github.com/alan-turing-institute/the-turing-way/blob/master/.github/ISSUE_TEMPLATE/ISSUE_TEMPLATE.md |
@sgibson91 I like that template! I agree that templates are helpful - I mostly just want to reduce the number of them! |
Totally agree with you there! |
So, you/we are proposing to have 3 types of issues: New Hub, Incident report, and New Issue (with some templating), and then we use labels for |
Well, actually looking at #595, do we need one for Debriefs? |
What if we took the following approach:
Here I think we will have to strike a balance between "having millions of issues" and "having issues that are extremely large and overly-scoped". But in my experience oftentimes small-ish tasks can often be categorized as part of broader efforts, so we should track them as-such rather than creating different issues for them. edit: we may also gain a better intuition for this balance as we try out the "Deliverables Backlog" approach described in 2i2c-org/team-compass#188 - I think that we will build over time a good feel for issues that are "too tiny to be on the deliverables backlog" This is also related to 2i2c-org/team-compass#188 |
That the difficult piece 😉 . |
@damianavila though in those cases, they could always open up a blank issue, no? I agree that we don't want to discourage the creation of issues in general, though we definitely do want to encourage greater signal to noise. This could also be something that more time spent in refinement etc could help with. Perhaps any new issue should include a |
Those are back holes 😉 .
I totally agree with that.
Yes... what I am saying above is trying to do the refinement after the issue is created and not ask for that at the time to create the issue (I see looking for a parent issue as a sort of refinement).
I would assume every new issue needs-triage (without the need of a label) until we know it is ready to live standalone (and maybe that is the time to assign a label) or as a task in a parent issue. |
So, how should we relate that to this PR? I agree with you that blank issues are basically black holes, though the template that this PR is removing is just this markdown: # Summary
<!-- What is the context needed to understand this task -->
so it feels like that is not a lot of structure anyway, and I'm not sure that we gain much by having a template for this. Does that make sense? |
I'm on the same page with @damianavila. Extra decisions to be made at issue creation time add impedence to issue creation, especially since often issues are created in a batch. We can add a 'needs-triage' label by default to all blank issues? |
I'm actually amazed that this template isn't something like
That's the kind of structure I'd want. Or the Turing Way template I linked above. |
How about we combine a few of the ideas above into the following issue templates:
We keep <!--
Please complete the following sections when you open an issue. You are encouraged to keep this top level comment box updated as we learn more about the issue. If you have write access to the repository please also assign the appropriate label (or labels) to your issue.
-->
### Summary
<!-- Please provide a detailed description of the change or addition you are proposing, or the question you're asking. Please provide as much context as possible and link to related issues and/or pull requests. If this issue is related to a bug, include information about how to reproduce it.
-->
*Lorem ipsum dolor sit amet, consectetur adipiscing.*
### Tasks to complete
<!-- We suggest using bullets (indicated by * or -) and filled checkboxes [x] here -->
- [ ] *Lorem ipsum dolor sit amet, consectetur adipiscing.*
- [ ] *Lorem ipsum dolor sit amet, consectetur adipiscing.*
---
### Updates
<!-- To avoid that others have to read through the full thread of comments, please update the initial issue with important updates (e.g. decisions taken) regularly. You can update the task list and summary above directly (this is encouraged!) or add new information below in this new section.
-->
*Lorem ipsum dolor sit amet, consectetur adipiscing.* I could see value in keeping a |
Hey all - I didn't want to lose track of this PR, so I've opened up a new PR to supercede this one. What do folks think? #619 |
Thanks for opening #619, @choldgraf! I'll close this in favor of that. |
Too many options here was a bit overwhelming for me. I
am not sure what a specific 'Task' label gives us over
just blank issues.