vote: Adding @darryk10 to rules approvers #247
Conversation
|
Big +1 from me! /hold until we reach the majority or a week has passed cc @falcosecurity/rules-maintainers |
|
+1 , thank you @darryk10 ! |
LucaGuerra
changed the title
|
PS this PR triggered a bug in the CI that validates rules. I will dig into it. It's unrelated to this PR, so please ignore it. |
|
LGTM label has been added. Git tree hash: 12a4ec6a0bd814c4114af74d1b626007e119303b
|
|
Big +1 from me! |
|
Big +1 great pleasure to welcome you as approver ❤️ |
|
+1 from me! Welcome!! |
poiana
added
dco-signoff: yes
size/XS
and removed
dco-signoff: no
do-not-merge/contains-merge-commits
size/M
labels
Added myself to maintainers for rules Signed-off-by: Stefano Chierici <stefano.chierici@sysdig.com>
There was a problem hiding this comment.
With 5 binding votes in favor and nobody against Majory vote has passed!
@darryk10 welcome onboard! 👏
/hold cancel
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: darryk10, Kaizhe, leogr 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:
Approvers can indicate their approval by writing |
|
LGTM label has been added. Git tree hash: b30d8cb409ada1fc977f8bf0ca95768acc9c3a31
|
What type of PR is this?
After more than a year as a reviewer in the falcosecurity/rules repo and more than 3 years as a contributor in the Falco community, I want to propose myself as a rule approver.
Any specific area of the project related to this PR?
Proposed rule maturity level
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer: