Skip to content

Commit 4b49b0e

Browse files
joshgavMylesBorins
authored andcommitted
doc: add CTC meeting minutes 2016-09-14
PR-URL: #8726 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 88b0067 commit 4b49b0e

File tree

1 file changed

+153
-0
lines changed

1 file changed

+153
-0
lines changed

doc/ctc-meetings/2016-09-14.md

+153
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,153 @@
1+
# Node Foundation CTC Meeting 2016-09-14
2+
3+
## Links
4+
5+
* **Audio Recording**: TBP
6+
* **GitHub Issue**: [#8506](https://github.com/nodejs/node/issues/8506)
7+
* **Minutes Google Doc**: <https://docs.google.com/document/d/1kW9wLdfojpIGbu3fQMrtr7S_RJFq8CWupoKnmtWw17Q>
8+
* _Previous Minutes Google Doc_: <https://docs.google.com/document/d/1GRgBC5Dv5luqlRc6egtxmkrQULJXa3M3CqGqWfoogV8>
9+
10+
## Present
11+
12+
* Bradley Meck @bmeck (observer/GoDaddy/TC39)
13+
* Сковорода Никита Андреевич @ChALkeR (CTC)
14+
* Evan Lucas @evanlucas (CTC)
15+
* Jeremiah Senkpiel @Fishrock123 (CTC)
16+
* Josh Gavant @joshgav (observer/Microsoft)
17+
* Julien Gilli @misterdjules (CTC)
18+
* Brian White @mscdex (CTC)
19+
* Jenn Turner @renrutnnej (observer/Node.js Foundation)
20+
* Rod Vagg @rvagg (CTC)
21+
* Seth Thompson @s3ththompson (observer/Google)
22+
* Trevor Norris @trevnorris (CTC)
23+
* Rich Trott @Trott (CTC)
24+
25+
## Standup
26+
27+
* Bradley Meck @bmeck (observer/GoDaddy/TC39)
28+
* Synched up with more people about ES Modules
29+
* Сковорода Никита Андреевич @ChALkeR (CTC)
30+
* Mostly nothing this week.
31+
* Evan Lucas @evanlucas (CTC)
32+
* Commented on issues/prs
33+
* Jeremiah Senkpiel @Fishrock123 (CTC)
34+
* v6.6.0 Release WIP
35+
* ES Modules discussions
36+
* Misc PRs / Issues
37+
* Async_Hooks review / Thinking
38+
* Josh Gavant @joshgav (observer/Microsoft)
39+
* Diag WG activity
40+
* modules discussion
41+
* Julien Gilli @misterdjules (CTC)
42+
* Timers code reviews and investigating/fixing libuv bugs on SmartOS. Read Async Hooks EP.
43+
* Brian White @mscdex (CTC)
44+
* Investigated UDP performance again for the purposes of a JS DNS resolver. With a few tweaks/optimizations, we can at least achieve performance very close to c-ares.
45+
* Reviewed PRs, commented on issues
46+
* Jenn Turner @renrutnnej (observer/Node.js Foundation)
47+
* Observing, no update.
48+
* Rod Vagg @rvagg (CTC)
49+
* Travel, administrative, board, TSC, meetings, yabba yabba
50+
* Seth Thompson @s3ththompson (observer/Google)
51+
* v8_inspector move to v8 is almost done, builds in tree now
52+
* continued discussion on microtask queue inspection here: https://bugs.chromium.org/p/v8/issues/detail?id=4643
53+
* Trevor Norris @trevnorris (CTC)
54+
* async_hooks. submitted PR and updated EP.
55+
* Rich Trott @Trott (CTC)
56+
* Onboarded @not-an-aardvark (with @addaleax’s help), @imyller, and @gibfahn
57+
* ESLint update (replaced a custom rule with a built-in), test PRs, doc PRs
58+
* A burst of first-time-contributor PRs landed, woo hoo
59+
60+
61+
## Agenda
62+
63+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
64+
65+
* AsyncWrap public API proposal [node-eps#18](https://github.com/nodejs/node-eps/pull/18)
66+
67+
68+
## Previous Meeting Review
69+
70+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
71+
72+
* src: add no-op for --harmony-proxies flag [#8395](https://github.com/nodejs/node/pull/8395)
73+
74+
* @ofrobots to float a patch on our branch of V8 5.1 and open a PR. Discussion to continue there.
75+
* Keep on agenda for next week just in case.
76+
77+
* AsyncWrap public API proposal [node-eps#18](https://github.com/nodejs/node-eps/pull/18)
78+
79+
* Everyone read [node-eps#18](https://github.com/nodejs/node-eps/pull/18).
80+
* Vote next week on Node-EP.
81+
* Open a PR for code in nodejs/node.
82+
83+
---
84+
85+
## Minutes
86+
87+
### AsyncWrap public API proposal [node-eps#18](https://github.com/nodejs/node-eps/pull/18)
88+
89+
PR: [#8531](https://github.com/nodejs/node/pull/8531)
90+
91+
@trott: move to vote on the EP. Anything to discuss first?
92+
93+
@rvagg: One thing Trevor adjusted since discussion - some desire to have hooks that execute on one execution branch. Trevor can describe.
94+
95+
@trevnorris: if you were to enable a hook, run an async op, then disable the hook, any ops that took place during time when it was enabled will continue to have hooks called for them even after it’s disabled.
96+
97+
This was okay, I could’ve managed it, but `nextTick` alone increases the amount of hooks to be processed 3x.
98+
99+
**Votes**
100+
101+
* Yes - 7 - @rvagg, @evanlucas, @Chalker, @Fishrock123, @trevnorris, @mscdex, @trott
102+
* No - 0
103+
* Abstain - 1 - @misterdjules
104+
105+
---
106+
107+
### Other
108+
109+
@Chalker: Perhaps we should discuss GitHub functionality changes, such as the review capability in the web interface.
110+
111+
@trott: Open a PR to discuss. It could make fixing up the commit message simpler.
112+
113+
---
114+
115+
## Q/A on public channels
116+
117+
Q: @williamkapke: Live CTC meeting in Austin? [tsc#137](https://github.com/nodejs/tsc/issues/137)
118+
119+
A: @rvagg: More people probably and a good time too.
120+
121+
Q: What’s the status of ES6 modules? What’s the team’s personal opinion on that?
122+
123+
@Fishrock123: Mostly kept up in Node-EP 002 and related PRs. Some of us and @bmeck will be going to TC39 meeting at end of Sept. and have materials prepared to discuss problems we’re facing.
124+
125+
@trott: My personal opinion is that I hope we can work with TC39 and get alterations such that we can avoid `.mjs` extension, but if that doesn’t happen the `.mjs` extension isn’t ideal but I do believe that it’s better than other options.
126+
127+
@rvagg: My personal opinion is that we are unfortunately trapped in a tough corner where TC39 has created a spec which has technical problems for Node and we’ve had difficulty getting them to hear our concerns. They’d like us to make changes on our side and that would be enough, but that’s quite complicated.
128+
129+
We have many users who like the idea of modules, but they’re actually transpiled versions and not implemented by browsers.
130+
131+
@trott: I’m optimistic that things will work out well. I have a lot of faith in Bradley, Jeremiah, et al.
132+
133+
@Fishrock123: No browser / VM has this fully implemented yet. V8 would have to implement it first and it definitely won’t be done for Node.js v7.
134+
135+
@bmeck: Chakra’s impl is basically done for `type=module` spec in the browser, but that would drastically change if we went to TC39 and they changed the behavior.
136+
137+
@Chalker: Edge has modules in a preview release.
138+
139+
@Fishrock123: That’s Chakra, yes.
140+
141+
@rvagg: There will be a talk at Node Interactive Austin by Bradley, you should attend there. (CTC call at the same time, we might need to think about whether to have CTC call that day.)
142+
143+
144+
## Upcoming Meetings
145+
146+
* CTC: 2016-09-21
147+
* TSC: 2016-09-22
148+
* Build: 2016-09-20
149+
* Diagnostics: 2016-09-21
150+
* Benchmarking: 2016-09-21? [Benchmarking#56](https://github.com/nodejs/benchmarking/issues/56)
151+
* LTS:
152+
* Post-Mortem:
153+
* API:

0 commit comments

Comments
 (0)