-
Notifications
You must be signed in to change notification settings - Fork 16
Conversation
Are there other labels we want/need? |
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.
A few tweaks (see inlined comments), but LGTM after that. Thanks!
- name: bug | ||
description: "Something isn't working" | ||
color: d73a4a | ||
- name: documentation |
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.
Hmm, let's drop this (like we did for etcd.io) -- since most PRs over a website repo will be for documentation (i.e., let's take that as the default when there's no extra label).
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.
I was thinking about that -- but as it is one of the default labels that GitHub creates on repo creation, I wonder if there is some value in keeping it (will template users expect it, even if we don't use it).
Let's start with this very nice set. |
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.
LGTM!
0ae4149
to
0e1f735
Compare
(latest force-push rebases the PR on main) |
Signed-off-by: Nate W <4453979+nate-double-u@users.noreply.github.com>
Signed-off-by: Nate W <4453979+nate-double-u@users.noreply.github.com>
0e1f735
to
81bbf2e
Compare
Nice, the existing label assignments were persevered. |
Fixes #79
Note: Adding labels this way will remove labels already added via the GitHub UI.
In case GitHub isn't able to reapply labels when merging this PR, this is how they are currently applied (to open issues/PRs):
bug: #56, #52, #45
cleanup: #85
documentation: #29, #24, #23, #22
enhancement: #83, #80, #28, #27, #26, #25, #21, #20, #19, #18
infrastructure: #85, #83, #79, #68
question: #76, #87