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

Improve Github labels for NuttX #13522

Open
1 task done
raiden00pl opened this issue Sep 18, 2024 · 2 comments
Open
1 task done

Improve Github labels for NuttX #13522

raiden00pl opened this issue Sep 18, 2024 · 2 comments

Comments

@raiden00pl
Copy link
Contributor

raiden00pl commented Sep 18, 2024

Description

Currently, the labels are not very readable. There is too much colors for lables that don't really mean anything and there is no distinction on the type/context of label.

Here are how it's organized in RIOT:

https://github.com/RIOT-OS/RIOT/labels

here for Zephyr:

https://github.com/zephyrproject-rtos/zephyr/labels

here for RT-Thread:

https://github.com/RT-Thread/rt-thread/labels

I would like to change it to be similar to the examples above:

  1. add context for some labels like: "Platform: arm", "Area: Documentation", "Community: help wanted".
    It would be ideal if each label had a context assigned, but I don't know if it's possible.
  2. change the color of labels so that labels with the same context have the same color.
  3. see what labels are missing (for example, the size of the change)

Does anyone have any objections ? Personally, I like how it's done in RIOT. If you know of any other github projects that have this handled nicely, please link here.

If we have clear and readable labels, we can later automate labeling with https://github.com/actions/labeler and other tools

Verification

  • I have verified before submitting the report.
@raiden00pl
Copy link
Contributor Author

mynewt uses https://github.com/marketplace/actions/pr-size-labeler-action to classify the PR size. This action also helps to limit the size of the PR so that huge changes will have to be split into smaller parts. We can use the size label for further conditions in CI.

@raiden00pl
Copy link
Contributor Author

raiden00pl commented Sep 19, 2024

Here PR with size labeler:

Here PR that adds specifier to labels:

Next steps:

  • modify github labels here: https://github.com/apache/nuttx/labels

  • initial support for https://github.com/actions/labeler to auto label PRs

  • [IDEA] most labels can be assigned automatically, but some labels require a deeper understanding of PR that only the author (or reviewer) has. For example PR type (bug fix, breaking change, enhancement, new feature) must be provided by a human (at least for now, LLM should be able to handle this too).
    For this reason, maybe we should force the PR author to select the "Type:" label. It's minimal amount of work and there's a greater chance that the author will click "breaking change" label when it's required which is commonly avoided now :)

  • port all changes to nuttx-apps repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant