|
| 1 | +# io.js TC Meeting 2015-03-18 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Public YouTube feed**: http://www.youtube.com/watch?v=tQwVcuYCiZ4 |
| 6 | +* **Google Plus Event page**: https://plus.google.com/events/cneon2drmol62u4drm8aegjnrkk |
| 7 | +* **GitHub Issue**: https://github.com/iojs/io.js/issues/1187 |
| 8 | +* **Original Minutes Google Doc**: https://docs.google.com/document/d/1It6PTEBQ7OjW8P88hCLoXvcWKA3Q8dY2eYEHIF6FvS4 |
| 9 | + |
| 10 | +## Agenda |
| 11 | + |
| 12 | +Extracted from https://github.com/iojs/io.js/labels/tc-agenda prior to meeting. |
| 13 | + |
| 14 | +* [#1134](https://github.com/iojs/io.js/pull/1134) Add Docker working group |
| 15 | +* [#1140](https://github.com/iojs/io.js/pull/1140) Revert stream base / @piscisaureus / merge policy questions |
| 16 | +* [#1130](https://github.com/iojs/io.js/pull/1130) Nominating Jeremiah Senkpiel @Fishrock123 to the TC |
| 17 | +* [#1077](https://github.com/iojs/io.js/pull/1077) Pass args to process.nextTick() / @trevnorris |
| 18 | + |
| 19 | +## Minutes |
| 20 | + |
| 21 | +### Present |
| 22 | + |
| 23 | +* Ben (TC) |
| 24 | +* Bert (TC) |
| 25 | +* Chris (TC) |
| 26 | +* Fedor (TC) |
| 27 | +* Trevor (TC) |
| 28 | +* Rod |
| 29 | + |
| 30 | +### Mini stand-up |
| 31 | + |
| 32 | +* Ben: reviewing bugs, fixing pull requests |
| 33 | +* Bert: debugging Windows issues (related to #1005), working on a more helpful CI dashboard |
| 34 | +* Chris: working on tools to inspect amount of ecosystem breakage that changes cause |
| 35 | +* Fedor: working through PayPal leak issue, working on net.socket() changes |
| 36 | +* Rod: .. |
| 37 | +* Trevor: misc PRs, process.nextTick(), timer bug |
| 38 | + |
| 39 | +### Review of last meeting |
| 40 | + |
| 41 | +* lib: fix process.send() sync i/o regression [#774](https://github.com/iojs/io.js/issues/774) / @bnoordhuis |
| 42 | +* iojs: introduce internal modules [#848](https://github.com/iojs/io.js/issues/848) / @chrisdickinson / @trevnorris / (@vkurchatkin) |
| 43 | +* url: significantly improve the performance of the url module [#933](https://github.com/iojs/io.js/issues/933) / @mikeal |
| 44 | +* Ship websockets in core [#1010](https://github.com/iojs/io.js/issues/1010) / @piscisaureus |
| 45 | +* RFC: upgrade to V8 4.2? [#1026](https://github.com/iojs/io.js/issues/1026) / @bnoordhuis |
| 46 | +* Colin resigning from TC [#1056](https://github.com/iojs/io.js/pull/1056) |
| 47 | + |
| 48 | +### [#1134](https://github.com/iojs/io.js/pull/1134) Add Docker working group |
| 49 | + |
| 50 | +* Rod: originally conceived to be a “sub-WG” of Build WG but this makes sense because there is little overlap with the Build team or skills required for the Build team. |
| 51 | +* Rod: this group is already maintaining official Docker images or io.js located at https://github.com/iojs/docker-iojs which are also the “official” Docker images for io.js. |
| 52 | + |
| 53 | +### [#1140](https://github.com/iojs/io.js/pull/1140) Revert stream base / @piscisaureus / merge policy questions |
| 54 | + |
| 55 | +* Bert: in favour of stream_base now and don’t want it reverted given the fixes that have been made. The real problem was the introduction of a major change that lead to a lot of breakage and a lot of fix commits. Proposed that we have a “ng” (or similar) branch with releases for testing before merging. |
| 56 | +* Chris: agree that we need a canary policy of some kind to deal with this |
| 57 | +* Bert: feature flags are a good way of dealing with this and letting them test changes |
| 58 | +* Ben: time-based releases like Rust and Chrome |
| 59 | +* Fedor: main problem is that we didn’t have enough tests for this feature when it was merged. |
| 60 | +* Rod: there’s awkwardness around releasing given that we only have one tip and we can’t cut patch releases once we have semver-minor changes are merged. |
| 61 | +* Trevor: should have semver-minor branches where patch fixes go in and minor changes go in to master/whatever. |
| 62 | +* Discussion about git process |
| 63 | +* Discussion about semver |
| 64 | +* Discussion about minor/patch and what it communicates |
| 65 | +* Discussion about test coverage |
| 66 | +* Rod asked OpenSSL fix: agreed to punt on a release until OpenSSL fix has been released and go immediately after that. |
| 67 | + |
| 68 | +### [#1130](https://github.com/iojs/io.js/pull/1130) Nominating Jeremiah Senkpiel @Fishrock123 to the TC |
| 69 | + |
| 70 | +* Rod: there was some off-record discussion about TC membership process related to #1130 and #1131. |
| 71 | + - Neither individual was asked and it would be good to at least have it passed by them first |
| 72 | + - Would be good to have a smooth on-boarding mechanism to bringing someone on to meetings before officially making them TC, agreed to invite Jeremiah to next TC _if_ he’s interested in being involved. |
| 73 | + - Generally agreed that increasing the size of the TC is a good thing (minus the meeting-overhead added by more people) |
| 74 | + |
| 75 | +### [#1077](https://github.com/iojs/io.js/pull/1077) Pass args to process.nextTick() / @trevnorris |
| 76 | + |
| 77 | +* Trevor: passing arguments / context through process.nextTick() in core is common and would be significantly faster if we use a setArgs() feature. |
| 78 | +* Chris: needed in readable-stream |
| 79 | +* Discussed API concerns and how underscored methods still become defacto in userland |
| 80 | +* Chris: could land the internal-module PR and use a Symbol for setArg |
| 81 | +* Trevor: major speed increase comes from flattening the call-structure, there’s no additional stack entries for passing arguments. Particularly in crypto, nextTick() is called a lot. |
| 82 | +* Rod: we need perf numbers from a user-experience perspective, perhaps that will make it more compelling to merge or perhaps it will make this look trivial that it’s not worth the cost of introducting ugly API |
| 83 | +* Trevor: will go off and come back with numbers |
| 84 | + |
| 85 | +### Major version bump |
| 86 | + |
| 87 | +* Ben: proposed making a new integration branch to merge all of the semver-minor features, plus a V8 upgrade, we can start pushing out nightly-style releases for playing around and punt the decision on _how_ to do a major release down the road a little. |
| 88 | +* Group agreed, Rod agreed to help come up with a release mechanism for nightlies to couple with this. |
| 89 | + |
| 90 | +### Next meeting |
| 91 | + |
| 92 | +* 25th March 2015 |
0 commit comments