Skip to content

Conversation

@mikeal
Copy link
Contributor

@mikeal mikeal commented Sep 21, 2015

This is the living document for the foundation's umbrella program for multiple top level projects.

Please review and provide feedback.

@mikeal
Copy link
Contributor Author

mikeal commented Sep 21, 2015

@nodejs/tsc

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is why we have collaboration WG, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the collaboration wg is there to share knowledge about how to best collaborate and to build tools that help us do so, the TC or WG responsible for each repo is ultimately the arbiter of conflicts. this line delegates that responsibility from the TSC (where it current resides because of the TSC Charter passed by the foundation's board of directors) to the Core TLD for the "node.js" project.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this section for? Feels a little incongruous.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was a placeholder and hasn't been updated because we all know what node.js core is ;)

@Fishrock123
Copy link
Contributor

@mikeal This doesn't need to mention how the tsc is governed because we have to adopt how it already is because of how the foundation as chartered, correct?

@mikeal
Copy link
Contributor Author

mikeal commented Oct 6, 2015

@Fishrock123 correct, the TSC Charter was already passed by the board and establishes the TSC and its governance. If this is passed by the TSC the authority over Core will be moved, by the TSC, from the TSC directly to the Core TLP, leaving the rest of the "technical side" of the foundation with the TSC. As such, this document describes the process by which TLPs get accepted and run under the TSC.

@jasnell
Copy link
Member

jasnell commented Oct 6, 2015

This looks fine overall. I'm a bit wary about attempting to solve too many problems before they're actually problems but in general this appears solid. Let's just make sure we leave it open for iteration once we start actually onboarding other projects.

@mhdawson
Copy link
Member

mhdawson commented Oct 6, 2015

Overall looks good to me but you probably have to update the list of TC members due to the recent changes

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: inconsistent word wrap. also, other places.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"project-specific"

@trevnorris
Copy link

Please, no spaces in file names.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is going to become redundant as soon as this is adopted. Perhaps this belongs under "History" or "Context" or something

@rvagg
Copy link
Member

rvagg commented Oct 14, 2015

"Top Level" should become "Top-level" (note the unfortunate lower-casing of the l, but that's just how it is) and "top level" should become "top-level"

@mikeal
Copy link
Contributor Author

mikeal commented Oct 14, 2015

Ok, tons of fixes are in. Pretty sure I handled all the current complaints but let me know if I missed any.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this say "Core working groups" and link to WORKING_GROUPS.md for 'the rest' ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's only the chartered WGs, there's a ton of un-chartered ones as well.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In a glass house? Or did you mean 'manner'?

README.md Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indent error? Or are "Mentors" and "Core TLP" subcategories?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The “Core TLP” is pretty much the whole “project” as it stands today, so it is its own category and will have its own WGs. I’m sending a small change to make that clearer.

On Oct 14, 2015, at 1:57PM, Ben Noordhuis [email protected] wrote:

In README.md #2 (comment):

+If your project is interested in joining the Node.js Foundation please read the [Project Lifecyle.md](./Project Lifecycle.md) documentation.
+
+## TSC Members
+
+## Top Level WG and TLPs
+
+* Working Groups

  • * Mentors
    +* Top Level Projects
  • * Core TLP
    Indent error? Or is "Core TLP" a subcategory?


Reply to this email directly or view it on GitHub https://github.com/nodejs/TSC/pull/2/files#r42051998.

@Fishrock123
Copy link
Contributor

TSC Voted in favor of this in nodejs/node#2795

mikeal added a commit that referenced this pull request Oct 19, 2015
@mikeal mikeal merged commit c22584d into master Oct 19, 2015
@Trott Trott removed the tsc-agenda label Sep 2, 2017
@sam-github sam-github deleted the lifecycle branch December 5, 2019 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.