RFC: Should we remove the 'good first issue' label?

Related to other discussions on low effort PRs #31679 and #32207

Should we remove the 'good first issue' label?

Note that we mention this label in the "[Issues for new contributors](https://github.com/numpy/numpy/pull/21355)" section of the docs. It is also mentioned briefly at the end of the "[A typical workflow for triaging issues](https://scikit-learn.org/dev/developers/bug_triaging.html#a-typical-workflow-for-triaging-issues)" section.

Reasons for:

* currently there are no open issues with this label (https://github.com/scikit-learn/scikit-learn/issues?q=is%3Aissue%20state%3Aopen%20label%3A%22good%20first%20issue%22)
* it is rare for good first issues to occur naturally. Often it requires maintainers specifically trying to think of one and spending time writing thorough details and guidance in an issue, probably what happens when searching for issues for people to work on in sprints, and even then, during the sprint, guidance is often required.
* it often attracts *many* low quality PRs - this is possibly because it's aggregated on sites such as https://goodfirstissue.dev/. It's difficult to find examples as we often remove the label but maybe #31318 and #30352 are examples of how much interest the label often attracts.
* it may be better for a user to start work on something that they use themselves and are familiar with, or something they already have background knowledge of. This would of course vary between people, making it a difficult label to use.
   * related, as described in @betatim's [post](https://discuss.scientific-python.org/t/what-does-it-mean-to-be-a-project-that-is-welcoming-to-newcomers/2127/3) on scientific-python, the type of users attracted to this label may be simply interested in getting a PR merged in a 'big' project. From Ralf: 'Promising new contributors are almost always motivated by a problem/topic, not by “getting on the scoreboard”'  
* for reference NumPy does not have a good first issue label (https://github.com/numpy/numpy/pull/21355) they do have a 'sprintable' label

This is just a request for comment, I understand if we decide to keep this label in the end.

cc @scikit-learn/core-devs

COMMENT:
I am fine with not using the "good first issue" label and update the doc about "new issues for contributors". Not sure whether this is really worth deleting the label.

I am a not sure how much a difference this will make, my feeling is that people (or maybe bots, who knows 😅) jump on seemingly random issues with "can I work on it, assign this to me" which I tend to be more and more grumpy about if I am being honest 😱.

I'd rather keep this kind of easy issues for face-to-face sprints, where is a lot easier to interact with new contributors. For completeness: we do have a "Sprint" label.

As a recent example of a "good first issue", the Cython relative import was fine IMO https://github.com/scikit-learn/scikit-learn/issues/32315. It helped attracting attention to it and completing it in a reasonable amount of time. Would I have done it quicker myself, maybe, but reviewing this kind of PRs was a no-brainer and my overall feeling is that it went smoothly.

Labels: RFC