-
-
Notifications
You must be signed in to change notification settings - Fork 69
administrative: team listed in README does not match team on GitHub #127
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
Comments
Is there any formal requirements and obligations to being part of the WG? So far I've been to two meetings, without explicitly being invited to. |
@hashseed I think adding yourself to the list in the README is as official as it gets and there's no further expectations beyond being active (https://github.com/nodejs/diagnostics/blob/master/GOVERNANCE.md#wg-membership). |
Some of the members are APM providers that just wants to be notified on relevant issues. So it is hard to judge activity. Also, there is no clear guideline of how much activity is needed to be a member. The README list is the most updated list we have. The reason why @nodejs/diagnostics is outdated is mostly that no one in the WG have permissions to make changes to @nodejs/diagnostics. |
Currently four people in @nodejs/diagnostics have those permissions: @ofrobots, @mhdawson, @Fishrock123, and @joshgav. If that's not sufficient, they or I can add others if there's a clear obvious choice for someone to maintain it. I'm also happy to make changes if I can get clear instructions as to who should be removed and who should be added. |
The team is just not up to date with the README.md which should be the reference. I agree that a PR to add yourself the README.md is current process as far as I know. The list through is pretty long and some rationalization might make sense before we update the team. @nodejs/automation it would be great if we had a bot to sync teams with lists in the README.md for each WG. That would keep things in sync and be one less thing somebody has to remember to change. |
One of thought is that we may want to take this opportunity to merge the merge the members from the post-mortem WG. |
@mhdawson That might actually fix nodejs/automation#17 as well...we would need to come up with a consistent format of contacts for different repo though. (Is the format thing under the scope of nodejs/admin?) |
I think having a consistent format defined in the admin repo makes sense, WGs can then "opt-in" if they want their team to be automatically maintained. (I'm guessing every will do that as opposed to having a different format :)) |
Is there anything left to be done here? |
Hi! It appears that the @nodejs/diagnostics team in the GitHub organization is somewhat out of date (probably should remove @mikeal and @joshgav, for example). Is the list in the README file in thie repository also out of date? Can we try to come up with a current canonical list and get it in both places?
The text was updated successfully, but these errors were encountered: