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

doc: add TSC meeting notes 2015-06-10 #1943

Merged
merged 1 commit into from
Jun 17, 2015

Conversation

piscisaureus
Copy link
Contributor

@misterdjules suggested that this should go into the convergence repo. Would that be better?

(if we land it here then it'll end up there eventually also)

### joyent/node
* Nominating new collaborators to this repository #25481
* Node should not automatically change rlimits #8704
* Rre-purpose previous joyent/node TSC meeting as LTS working group
Copy link
Contributor

Choose a reason for hiding this comment

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

Rre typo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@rvagg
Copy link
Member

rvagg commented Jun 11, 2015

I'd prefer them to go in here but @Fishrock123 told me in my last one that they should be put into the iojs repo because stuff gets upstreamed from there.

Also, check your markdown, you either need dot-points or double-newlines to prevent the blocks of text from becoming paragraphs (I usually run markdown docs through gfm2html before a PR to see what it looks like).

@Fishrock123
Copy link
Contributor

Yes, it's probably best to land them here (in io.js) for now. You could technically land something like this into master-with-merges on nodejs/node, but it would be a bit weird and wouldn't be visible for a while still.

* Bert Belder (TSC)
* Ben Noordhuis (TSC)
* Julien Gilli (TSC)
* Mikeal Rogers (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.. technically incorrect as of yet (i.e. Mikeal is not currently TSC yet)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed. Also removed the TSC label from Domenic.

@piscisaureus
Copy link
Contributor Author

Can I get a final LGTM from someone?

@rvagg The markdown comes out just fine on my end. Can you point out where you're seeing a problem?

* TSC needs to elect a board representative. #1697

### joyent/node
* Nominating new collaborators to this repository #25481
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a blank line here?

@bnoordhuis
Copy link
Member

LGTM with some nits.

@rvagg
Copy link
Member

rvagg commented Jun 17, 2015

@piscisaureus I see dot-points now so no problem

PR-URL: nodejs#1943
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
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.

5 participants