-
Notifications
You must be signed in to change notification settings - Fork 42
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
Added CODEOWNERS and MAINTAINERS files #272
Added CODEOWNERS and MAINTAINERS files #272
Conversation
Signed-off-by: Toddy Mladenov <[email protected]>
Signed-off-by: Toddy Mladenov <[email protected]>
Signed-off-by: Toddy Mladenov <[email protected]>
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.
In Maintainers file Rakesh Gokarn [email protected] (@rgnote) need to be changed to Rakesh Gariganti [email protected] (@rgnote)
Signed-off-by: Toddy Mladenov <[email protected]>
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #272 +/- ##
==========================================
- Coverage 73.53% 73.24% -0.29%
==========================================
Files 23 23
Lines 1923 1985 +62
==========================================
+ Hits 1414 1454 +40
- Misses 410 429 +19
- Partials 99 102 +3
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
LGTM
Linking the issue #249 |
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.
LGTM but IANAM
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.
LGTM but IANAM
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.
LGTM
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.
LGTM but IANAM
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.
LGTM
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.
LGTM but IANAM
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.
LGTM but IANAM
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.
LGTM
Welcome new maintainers!
Signed-off-by: Toddy Mladenov [email protected]
This PR combines all the changes from the following PRs for updating the maintainers for notaryproject/notation-go sub-project:
Add Junjie Gao #265
Add Milind Gokarn #259
Add Patrick Zheng #264
Add Pritesh Bandi #258
Add Rakesh Gariganti #260
Add Shiwei Zhang #263
The proposal is to abandon the above PRs and merge only this one.