cleanup(OWNERS): remove inactive approvers #360
Conversation
poiana
added
kind/cleanup
dco-signoff: yes
kind/documentation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jasondellaluce The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
OWNERS
Outdated
| - leodido | ||
| - fntlnz | ||
| - kris-nova | ||
| - nibalizer |
falco/OWNERS
Outdated
| reviewers: | ||
| - bencer | ||
| - nestorsalceda | ||
| emeritus_approvers: | ||
| - bencer | ||
| - nestorsalceda |
|
and will need to bump the chart version, then i recommend to split in two prs for each chart |
|
ops, sorry pressed the wrong button |
Signed-off-by: Jason Dellaluce <jasondellaluce@gmail.com>
Co-authored-by: cpanato <ctadeu@gmail.com> Signed-off-by: Jason Dellaluce <jasondellaluce@gmail.com>
Co-authored-by: cpanato <ctadeu@gmail.com> Signed-off-by: Jason Dellaluce <jasondellaluce@gmail.com>
jasondellaluce
force-pushed
the
cleanup/inactive-owners
branch
from
6496e2a to
58ea577
Compare
Signed-off-by: Jason Dellaluce <jasondellaluce@gmail.com>
There was a problem hiding this comment.
Please restore fntlnz https://github.com/falcosecurity/falco/issues/2115#issuecomment-1176045086
@cpanato I think you have a good point here, specially considering that the Falco chart has been recently released and deeply refactored. I'm gonna close this PR and split the OWNERS review in two: falcosecurity/falco#368, falcosecurity/falco#369 |
What type of PR is this?
/kind cleanup
/kind documentation
Any specific area of the project related to this PR?
What this PR does / why we need it:
For falcosecurity/evolution#157, this PR moves inactive approvers (who had very little or zero contributions over the past 6 months) from
approverstoemeritus_approversin OWNERS.Which issue(s) this PR fixes:
Special notes for your reviewer: