search

kubernetes-sigs / apisnoop

⭕️Snooping on the Kubernetes OpenAPI communications
https://apisnoop.cncf.io
Apache License 2.0
88 stars 42 forks source link

update OWNERS and add a NOTICE for cla exceptions #849

Closed MadhavJivrajani closed 9 months ago

MadhavJivrajani commented 9 months ago

Finishing setup of the repo Ref https://github.com/kubernetes/org/issues/4705

  1. Updating OWNERS files to reflect folks who are part of the GitHub orgs
  2. Adding a NOTICE file for folks who have not signed the CNCF CLA as per guidelines

/hold

netlify[bot] commented 9 months ago

Deploy Preview for apisnoop canceled.

Name Link
Latest commit be9354507ceda8782349a22606fc5d5fc4ecf42d
Latest deploy log https://app.netlify.com/sites/apisnoop/deploys/65d2e4c3dc37c500081f8415
MadhavJivrajani commented 9 months ago

Please hold off on merging this PR till we can remove the hold label.

Priyankasaggu11929 commented 9 months ago

Both NOTICE & OWNERS file changes looks good to me.

/lgtm /approve

MadhavJivrajani commented 9 months ago

/assign @BobyMCbobs for approval

BobyMCbobs commented 9 months ago

@MadhavJivrajani, thank you!

/approve /lgtm

k8s-ci-robot commented 9 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: BobyMCbobs, MadhavJivrajani, Priyankasaggu11929

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubernetes-sigs/apisnoop/blob/main/OWNERS)~~ [BobyMCbobs] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
MadhavJivrajani commented 9 months ago

/hold cancel