-
-
Notifications
You must be signed in to change notification settings - Fork 777
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
Roll Out plan: Make issues required to change all the places that Good Second Issues appears with Good First Issue #4432
Comments
@jdingeman Please review this and its related issues and see if these are all the steps required for the roll out plan. Or is there something I have forgotten about.
|
Hi @ExperimentsInHonesty, this looks good and is very thorough. The timing will be a little funny (of course, it's never straightforward lol), but I think #4423 will technically need to happen first before the other issues actually start getting created. I don't think the order matters so much except that #4423 happens first, but I think it'll get too messy with an issue writing party because of how different the issues might be and it may just be easier to write them up all at once when the places are all found. |
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Please add update using the below template (even if you have a pull request). Afterwards, remove the '2 weeks inactive' label and add the 'Status: Updated' label.
If you need help, be sure to either: 1) place your issue in the You are receiving this comment because your last comment was before Tuesday, October 8, 2024 at 12:05 AM PST. |
Please add update using the below template (even if you have a pull request). Afterwards, remove the '2 weeks inactive' label and add the 'Status: Updated' label.
If you need help, be sure to either: 1) place your issue in the You are receiving this comment because your last comment was before Tuesday, October 15, 2024 at 12:04 AM PST. |
Dependency
Issues Created
Overview
We need to create a plan and issues for changing from
good second issue
label togood first issue
label, so that it all gets rolled out at onceAction Items
good first issue
label instead of theComplexity: Good second issue
Onboarding infos/links
column (there could be two tasks here, once to add a card that explains the change and changing the card(s)Making the label change on issues Replace all "good second issue" labels with "good first issue" label on open PRs and Issues #4431 and adding a note to any issue that has an assignee.No open PRs or Issues withgood second issue
so 4431 was closed.good first issue
label instead of theComplexity: Good second issue
Resources/Instructions
Dependency for all the issues
Message that should go on PR issue. See issue 4423 for example
The text was updated successfully, but these errors were encountered: