-
Notifications
You must be signed in to change notification settings - Fork 110
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
feat: add missing CODEOWNERS and MAINTAINERS file #635
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #635 +/- ##
==========================================
+ Coverage 70.51% 72.90% +2.38%
==========================================
Files 10 10
Lines 2123 1657 -466
==========================================
- Hits 1497 1208 -289
+ Misses 505 328 -177
Partials 121 121 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Marvin Drees <[email protected]>
- rdimitrov | ||
- kommendorkapten | ||
- mnm678 | ||
- znewman01 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think @znewman01 left as a maintainer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Zack should say something if that is not the case, but he is still also in the @theupdateframework/go-tuf-maintainers list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, I just copied who was in the maintainers list in the org. If that info was outdated we should both change it there and in this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that makes sense, we should use that list as authoritative.
No description provided.