search

star-bnl / star-sw

Core software for STAR experiment
26 stars 62 forks source link

chore: remove unresponsive reviewers from CODEOWNERS #691

Open plexoos opened 5 months ago

plexoos commented 5 months ago

As discussed, we've cleaned up the CODEOWNERS file by removing unresponsive or inactive members.

plexoos commented 5 months ago

Feel free to submit your suggestions or commit to this branch directly before it's merged.

plexoos commented 5 months ago

Frank, was it you who added ZaochenYe? There is no such github user

plexoos commented 5 months ago

Also, I don't think @ideppner has ever accepted the invitation to the star-bnl organization. Should be removed?

fgeurts commented 5 months ago

@rkunnawa is not in STAR anymore. I am not sure who @zhux97. Cannot find a profile on GitHub.

fgeurts commented 5 months ago

Frank, was it you who added ZaochenYe? There is no such github user

Zaochen's GitHub ID is @Morning-Ye

starsdong commented 5 months ago

@rkunnawa is not in STAR anymore. I am not sure who @zhux97. Cannot find a profile on GitHub.

@zhux97 is Xianglei Zhu.

plexoos commented 5 months ago

I'm fine with keeping anyone who is willing to own the code and monitor changes in the CODEOWNERS file. My understanding was that the goal was to reduce the list of automatically requested reviewers to streamline the approval process and speed up merging. If someone wants to be notified about specific activities, they can adjust their notification settings or star the repository. More details can be found here: GitHub Notification Settings.

fgeurts commented 5 months ago

It may be good to bring that to Victor’s attention. We all agree that his expert opinion will always be much appreciated!

-f.

On Jun 26, 2024, at 5:30 PM, Dmitri Smirnov @.***> wrote:

I'm fine with keeping anyone who is willing to own the code and monitor changes in the CODEOWNERS file. My understanding was that the goal was to reduce the list of automatically requested reviewers to streamline the approval process and speed up merging. If someone wants to be notified about specific activities, they can adjust their notification settings or star the repository. More details can be found here: GitHub Notification Settings https://docs.github.com/en/account-and-profile/managing-subscriptions-and-notifications-on-github/setting-up-notifications/configuring-notifications#choosing-your-notification-settings.

— Reply to this email directly, view it on GitHub https://github.com/star-bnl/star-sw/pull/691#issuecomment-2192725319, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEMNVR7XY6LFE7MTUMIZVXTZJM6I5AVCNFSM6AAAAABJ6LEK22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJSG4ZDKMZRHE. You are receiving this because your review was requested.