Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

doc: add TC meeting 2015-05-13 minutes #1700

Merged
merged 1 commit into from
Jun 23, 2015

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented May 14, 2015

No description provided.

@mscdex mscdex added the doc Issues and PRs related to the documentations. label May 14, 2015
* Brian: looking over potential optimizations in the JS codebase, started working on a DNS resolver as a potential replacement for cares
* Chris: fixing race conditions in the REPL, poking at adding Ctrl-R history searching to readline
* Domenic: working with V8 team in Munich, working on v8-extras feature, putting large portions of things into snapshot to speed up startup and other things: https://groups.google.com/forum/#!topic/v8-users/D6FmTwlvCgk
* Jeremiah: issue management, working on tooling for automatic dependency upgrades, see [#1688](https://github.com/iojs/io.js/pull/1688)
Copy link
Contributor

Choose a reason for hiding this comment

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

My own wording here isn't really the best, it's not really automatic...

@Fishrock123
Copy link
Contributor

comment / question, but LGTM otherwise.


### Put `*Sync` methods behind a flag in some future major version [#1665](https://github.com/iojs/io.js/issues/1665)

* Ben: some people feel that fs.*Sync() methods are harmful and would like to see them go away and be behind a flag
Copy link
Member

Choose a reason for hiding this comment

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

The markdown parser gets confused by the lone asterisk here.

@bnoordhuis
Copy link
Member

LGTM with a style quibble.

@rvagg rvagg force-pushed the tc-minutes-2015-05-13 branch from 536c3ad to 24bc904 Compare June 23, 2015 09:56
PR-URL: nodejs#1700
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@rvagg rvagg force-pushed the tc-minutes-2015-05-13 branch from 24bc904 to 1330ee3 Compare June 23, 2015 09:58
@rvagg rvagg merged commit 1330ee3 into nodejs:master Jun 23, 2015
@rvagg rvagg mentioned this pull request Jun 30, 2015
mscdex pushed a commit to mscdex/io.js that referenced this pull request Jul 9, 2015
PR-URL: nodejs#1700
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants