-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
meta: move WORKING_GROUPS.md to CTC repo #11555
Conversation
Do we have a criteria for what belongs there and what belongs here? I would imagine the main criterion would be "Where will someone who is looking for X expect to find it?" With CTC meeting notes and agendas, the answer might be the CTC repo. But for a list of working groups? I'd look in the main repo. A lot of new contributors aren't even aware the CTC exists, much less the CTC repo. It would be a shame to move this information to a lower-visibility area like that. Also, having it there suggests that these are CTC working groups instead of Node.js working groups. |
Nope, no such criteria exists in any documented manner.
The first sentence of the working groups page also suggests that these are CTC working groups: Specifically:
For me, this falls under the same category of information as the list of dependent/adjacent projects in the TSC repo. These are formed by petitioning the CTC, they are chartered by the CTC and the information makes sense to me to keep there. That said, if it's at all controversial I'm not going to insist on it by any means, mainly just doing a bit of housekeeping. |
I think if this document was linked to from the |
@Trott ... so I'm clear, are you against moving this? |
@jasnell I'm not against it if we have a link in the nodejs/node README or somewhere else sensible/findable. |
2cd132b
to
96b1ec9
Compare
Link to the new location added to the readme |
PR-URL: #11555 Ref: nodejs/CTC#73 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
Landed in 06942d8 |
PR-URL: #11555 Ref: nodejs/CTC#73 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
PR-URL: #11555 Ref: nodejs/CTC#73 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
PR-URL: #11555 Ref: nodejs/CTC#73 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
PR-URL: nodejs/node#11555 Ref: nodejs/CTC#73 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
Move the working groups doc the CTC repo, where it more likely belongs...
Ref: nodejs/CTC#73
Checklist
Affected core subsystem(s)
meta