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

Update CODEOWNERS #90

Merged
merged 2 commits into from
Aug 2, 2022
Merged

Update CODEOWNERS #90

merged 2 commits into from
Aug 2, 2022

Conversation

jfrerich
Copy link
Contributor

Summary

This PR will remove myself as the codeownwer which gets added to every new PR. I wouldn't mind getting added as a reference for the plugin, but should be removed from being codeownwer.

Ticket Link

@jfrerich jfrerich requested a review from catalintomai July 24, 2022 00:59
@jfrerich jfrerich self-assigned this Jul 24, 2022
@jfrerich jfrerich added the 2: Dev Review Requires review by a core committer label Jul 24, 2022
@jfrerich jfrerich requested a review from mickmister July 24, 2022 01:00
@mickmister
Copy link
Contributor

@jfrerich We shouldn't leave the code owner blank. Either @mickmister, @catalintomai, or @levb should be the owner. I'll volunteer but will wait to see what others say

@jfrerich
Copy link
Contributor Author

@jfrerich We shouldn't leave the code owner blank. Either @mickmister, @catalintomai, or @levb should be the owner. I'll volunteer but will wait to see what others say

Thanks! I also would like to remove myself from
https://github.com/mattermost/mattermost-plugin-welcomebot/blob/master/CODEOWNERS
https://github.com/mattermost/mattermost-plugin-agenda/blob/master/CODEOWNERS

@catalintomai
Copy link

@jfrerich - please make me the codeowner for all three plugins. Thanks!

@mickmister mickmister merged commit 1a6cf66 into master Aug 2, 2022
@mickmister mickmister deleted the jfrerich-patch-1 branch August 2, 2022 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core committer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants