Skip to content
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

chore: update CODEOWNERS #6173

Merged
merged 1 commit into from
Feb 21, 2024
Merged

chore: update CODEOWNERS #6173

merged 1 commit into from
Feb 21, 2024

Conversation

knqyf263
Copy link
Collaborator

@knqyf263 knqyf263 commented Feb 21, 2024

Description

This PR includes the following four changes regarding code owners:

@josedonizetti is now maintaining Tracee, while @DmitriyLewen and @nikpivkin have been active in these areas for a long time and are sufficiently knowledgeable.

I originally founded Trivy with philosophy in mind. I've also been a code owner in each domain, like misconfiguration and k8s, to ensure consistency with other features. There have actually been several incidents of accidental breakage of core functions. However, as the project matured and the scope of Trivy expanded, my oversight became broader, leading to a bottleneck and slowing down development. Therefore, I am delegating each area to the respective code owners to prevent delays in development. Of course, I may still participate in discussions about significant feature additions or designs, but ultimately, the code owners can make the final decisions.

If it includes changes outside of each area, I will also review it, otherwise each code owner is free to merge PRs.

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@knqyf263 knqyf263 self-assigned this Feb 21, 2024
@knqyf263
Copy link
Collaborator Author

I'd wait for @simar7.

@simar7
Copy link
Member

simar7 commented Feb 21, 2024

I'd wait for @simar7.

Thank you for waiting 🙇‍♂️

@knqyf263 knqyf263 added this pull request to the merge queue Feb 21, 2024
Merged via the queue into aquasecurity:main with commit 19495ba Feb 21, 2024
13 checks passed
@knqyf263 knqyf263 deleted the code_owners branch February 21, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants