-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Add an explanation of the labels #4480
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
r? @matklad |
Wow, that is much more thorough explanations than I had expected! Thanks a lot @carols10cents , I'be definitely learned more about our issues now ❣️ @bors r+ |
📌 Commit 0cfe635 has been approved by |
Add an explanation of the labels As requested @matklad! ❤️ Are there any questions you have about the labels that remain unanswered?
☀️ Test successful - status-appveyor, status-travis |
@carols10cents what does
Specifically, I've just created #4563 and I think it's mentor in the 2) sense, but not in the 1), so I don't know if I should apply the label :) |
@matklad to me, it means at least 2) and optionally 1) if 2) is insufficient :) I don't know if my interpretation is common though! |
As requested @matklad! ❤️ Are there any questions you have about the labels that remain unanswered?