community
community copied to clipboard
tektoncd
[TEP Process] Add "Collaborators" option
This commit adds the ability to specify "collaborators" for a TEP. The intention of this commit is to recognize the efforts of contributors who take the time to review and understand a proposed design, or contributors who may have worked on part of the design but are not owners of the entire design.
do we want to add a "reviewers" option too? or is reviewing considered collaborating? either way, the incremental improvements are great!
/approve
hold until other governing board members approve changes to TEP processes /hold
cc @priyawadhwa @vdemeester @afrittoli @dibyom
do we want to add a "reviewers" option too?
Good question! I view a collaborator as a bit different from a reviewer, so it could make sense to have them as separate categories, although there's a lot of potential for overlap. I'm tempted to say let's stick with just collaborator for now, but I don't have a strong opinion here.
@bendory: changing LGTM is restricted to collaborators
In response to this:
/lgtm
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: afrittoli, dibyom, jerop, priyawadhwa, vdemeester
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~teps/OWNERS~~ [afrittoli,dibyom,jerop,priyawadhwa,vdemeester]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment