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.

Sign up for GitHub

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

Jump to bottom

Improve labeling system - refactor labeling logic #5916

Merged
merged 16 commits into from
Nov 19, 2023
Merged

Improve labeling system - refactor labeling logic #5916

merged 16 commits into from
Nov 19, 2023

Conversation

igorpecovnik
Copy link
Member

@igorpecovnik igorpecovnik commented Nov 15, 2023 •
edited
Loading

Description

Better labeling system and clear instructions should improve chaos in PR / issues.

Adding label table from YAML file.

@igorpecovnik igorpecovnik added Work in progress Unfinished / work in progress Discussion Being discussed - Voice your opinions :) Help needed We need your involvement labels Nov 15, 2023
@igorpecovnik igorpecovnik changed the title Add label table Improve labeling system - design issue and PR labeling logic Nov 15, 2023
@igorpecovnik igorpecovnik changed the title Improve labeling system - design issue and PR labeling logic Improve labeling system - refactor issue and PR labeling logic Nov 15, 2023
@github-actions github-actions bot added size/medium PR with more then 50 and less then 250 lines and removed size/M labels Nov 15, 2023
@igorpecovnik igorpecovnik removed the Discussion Being discussed - Voice your opinions :) label Nov 15, 2023
@igorpecovnik igorpecovnik changed the title Improve labeling system - refactor issue and PR labeling logic Improve labeling system - refactor labeling logic Nov 15, 2023
@igorpecovnik igorpecovnik added the 11 Milestone: Fourth quarter release label Nov 15, 2023
@igorpecovnik igorpecovnik removed Work in progress Unfinished / work in progress Help needed We need your involvement labels Nov 15, 2023
@igorpecovnik igorpecovnik added the Needs review Seeking for review label Nov 15, 2023
@igorpecovnik igorpecovnik force-pushed the igorpecovnik-patch-1 branch 9 times, most recently from 80f7721 to f4d125f Compare November 16, 2023 18:13
viraniac
viraniac previously approved these changes Nov 18, 2023
View reviewed changes
Copy link
Collaborator

@viraniac viraniac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

EvilOlaf
EvilOlaf previously approved these changes Nov 18, 2023
View reviewed changes
@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge and removed Needs review Seeking for review labels Nov 19, 2023
@igorpecovnik igorpecovnik merged commit 70832ae into main Nov 19, 2023
11 checks passed
@igorpecovnik igorpecovnik deleted the igorpecovnik-patch-1 branch November 19, 2023 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EvilOlaf EvilOlaf EvilOlaf left review comments

@viraniac viraniac viraniac left review comments

@littlecxm littlecxm Awaiting requested review from littlecxm littlecxm is a code owner

@TRSx80 TRSx80 Awaiting requested review from TRSx80 TRSx80 is a code owner

@rpardini rpardini Awaiting requested review from rpardini rpardini is a code owner

@hzyitc hzyitc Awaiting requested review from hzyitc hzyitc is a code owner

Assignees
No one assigned
Labels
11 Milestone: Fourth quarter release Ready to merge Reviewed, tested and ready for merge size/medium PR with more then 50 and less then 250 lines
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@igorpecovnik @EvilOlaf @viraniac