Skip to content
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

The GitHub Label Cleanup Proposal #3920

Closed
Hundrec opened this issue Dec 9, 2024 · 6 comments
Closed

The GitHub Label Cleanup Proposal #3920

Hundrec opened this issue Dec 9, 2024 · 6 comments
Assignees
Labels
status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: github Relates to the management of issues and pull requests on GitHub.

Comments

@Hundrec
Copy link
Collaborator

Hundrec commented Dec 9, 2024

Overview

The labeling system in this repository has become more and more useful since 0.3.0 was released.
For example, the status and type labels are now automatically applied upon issue creation.
However, a few inconsistencies have arisen in the past few months as new labels and GitHub actions have been implemented.

In this issue, I will detail actions that can be taken to organize and standardize labels to improve your workflow (especially with over 600 open issues!)

Important

Some of these changes will require edits to files such as label-actions.yml. I have created an associated PR: #3918.

Delete or Replace

Some labels duplicate the function of existing labels, or should be replaced with a better label.

Label Action
Bug Delete
Fixed Interally Delete
duplicate Delete
potential duplicate Delete
Needs R&D Replace with status: needs r&d

Categorize

The full list of labels is currently alphabetically disorganized. Sorting them into categories can clean this list up.

Category Label New Name PR
Size small size: small #3918
Size medium size: medium #3918
Size large size: large #3918
Topic help wanted topic: help wanted
Topic mods topic: mods
Topic platform specific topic: platform specific
Topic polish topic: polish
Topic scoring topic: scoring
Topic chart editor topic: chart editor
PR haxe pr: haxe
PR documentation pr: documentation

Color

Currently, Label Actions closes issues as not planned or complete for three status labels.

Just like the label status: rejected, highlighting these auto-close labels with prominent colors will serve two functions:

  • Allow devs to easily spot and apply labels that close the issue in the desired manner.
  • Allow community members to easily spot labels that indicate resolution.
Label Current Color New Color Purpose
status: needs r&d Orange #1B3A4D Group with status
status: resolved #1B3A4D #238636 Indicate resolution and auto-close
status: accepted #1B3A4D #238636 Indicate resolution
status: stale #1B3A4D #71383A Indicate auto-close
status: duplicate #1B3A4D #71383A Indicate invalid issue
low priority #ffffff #c8c8c8 White should be reserved for automatically applied PR labels

Additional color changes for topic: labels are listed below.

Label Current Color New Color
topic: help wanted #0e8a16 #FCF400
topic: mods #0e8a16 #c24b99
topic: platform specific #0e8a16 #c24b99
topic: polish #0e8a16 #c24b99
topic: scoring #0e8a16 #c24b99
topic: chart editor #FCF400 #c24b99

Tip

Refer to the table below to see which labels close issues when applied.

Label Close Reason
status: resolved Completed
status: rejected Not planned
status: stale Not planned
status: duplicate (no longer auto-closes)

Create

Our current labels don't quite cover the full range of issues and PR types. We can add some labels to help!

Label Color Description
pr: github #ffffff PR modifies GitHub files.
type: github #CA995E Relates to the management of issues and pull requests on GitHub.
type: chore #CA995E Involves minor adjustments that do not make functional changes.
type: security vulnerability #CA995E Involves a security vulnerability within the game.

Descriptions

Some labels do not currently have descriptions. Assuming all changes before this section have been implemented, the labels below should have their descriptions revised.

Label Proposed Description
status: resolved internally Completed development and will be released some time in the future.
pr: haxe PR modifies game code.
pr: documentation PR modifies documentation or README files.
topic: mods Related to the use of mods.
topic: chart editor Related to the operation of the Chart Editor.
topic: polish Involves minor polish to the UI or gameplay.
topic: scoring Involves the game's ranking and scoring systems.
topic: platform specific Specific to only one or several of the game's supported targets.
type: documentation Involves improvements or additions to documentation.
type: enhancement Involves an enhancement or new feature.
size: small A small pull request with 10 or fewer changes.
size: medium A medium pull request with 100 or fewer changes.
size: large A large pull request with more than 100 changes.

Additional Description Revisions

Label Proposed Description
status: needs r&d Requires further research and development.
status: pending triage Awaiting review.
status: pending pull request Dependent on a pull request that is currently awaiting review.
status: planned Planned to be implemented some time in the future.
status: reviewing internally Under consideration and testing.
status: needs clarification Requires more info from the author.
status: resolved The issue has been fixed or the suggestion has been implemented.

funkin.assets

The funkin.assets repo should also be revamped in a similar fashion.
All labels from the main repo should be transferred except the ones below:

Label Reason
type: compilation help Only applies to issues
status: resolved Only applies to issues
status: resolved internally Only applies to issues
high priority Historically only applied to issues (keep if useful for PRs)
topic: scoring Possibly not relevant to assets repo (keep if useful)

Closing

I believe that organization is essential for an improved workflow on GitHub.
Thank you for your time and consideration!

@AbnormalPoof
Copy link
Collaborator

AbnormalPoof commented Dec 11, 2024

This seems great for the most part, but I do feel like some labels (status: needs r&d, topic: help wanted, low priority) could be part of a miscellaneous category of some sorts (something like tag: or misc: ?).

@Hundrec
Copy link
Collaborator Author

Hundrec commented Dec 11, 2024

Good idea! I also felt like rounding up those labels would complete the alphabetization, but wasn't sure what to call the new category.

@EliteMasterEric
Copy link
Member

EliteMasterEric commented Jan 21, 2025

Hey @Hundrec and @AbnormalPoof do you have Discord? I think we'd be able to coordinate stuff a bit better that way.

@Hundrec
Copy link
Collaborator Author

Hundrec commented Jan 21, 2025

Yeah! Add me @Hundrec and I'll reply when I get home :)

@AbnormalPoof
Copy link
Collaborator

I'm only just now seeing this at 6 AM...

But yeah! My Discord username is kagaminerinlen.

@Hundrec Hundrec added status: reviewing internally Under consideration and testing. and removed status: pending triage Awaiting review. labels Jan 23, 2025
@EliteMasterEric EliteMasterEric self-assigned this Jan 23, 2025
@Hundrec Hundrec added status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: reviewing internally Under consideration and testing. labels Jan 23, 2025
@EliteMasterEric
Copy link
Member

All these changes are approved and done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: github Relates to the management of issues and pull requests on GitHub.
Projects
None yet
Development

No branches or pull requests

3 participants