-
Notifications
You must be signed in to change notification settings - Fork 677
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
New labels to reflect the state of PR #201
Comments
AFAIK: from our side not all developers can modify labels as they don't have enough permission to do so. |
@galpeter I think that this formality can be solved |
@galpeter btw, do all developers from your side have rights to change the assignee? |
@egavrin, no they can not modify any of the labels/milestone/assignee fields, they only can update their own PR/issue's title/description. |
@Seo-Young what do you think about this? |
@egavrin I think your suggestion on the labels are helpful to check the status of patches. Do you need my opinion on the modification rights on the labels/milestone/assignee fields? |
@Seo-Young thank you! How can we enable Szeged to change or modify this fields in pr? |
@egavrin Reviewers already has the rights for that. |
@Seo-Young I'd partly agree. Giving such rights to all developers is not feasible - I agree. But Szeged team may have such rights without any problems. |
@egavrin @Seo-Young We'd better discussing it later. We have to consider the community operating after open source release. |
We may introduce the following labels to reflect the state of PR:
waiting-for-review
waiting-to-be-fixed
make-push
orOK-to-commit
I think it can be useful, both for developers and reviewers.
The text was updated successfully, but these errors were encountered: