Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit e9c6aff

Browse filesBrowse files
joshgavMyles Borins
authored andcommitted
doc: add 2016-10-19 CTC meeting minutes
PR-URL: #9193 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent c1e5e66 commit e9c6aff
Copy full SHA for e9c6aff

File tree

Expand file treeCollapse file tree

1 file changed

+202
-0
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+202
-0
lines changed
Open diff view settings
Collapse file

‎doc/ctc-meetings/2016-10-19.md‎

Copy file name to clipboard
+202Lines changed: 202 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,202 @@
1+
# Node Foundation CTC Meeting 2016-10-19
2+
3+
## Links
4+
5+
* **Audio Recording**: TBP
6+
* **GitHub Issue**: [#9143](https://github.com/nodejs/node/issues/9143)
7+
* **Minutes Google Doc**: <https://docs.google.com/document/d/18rnF8IfO10aQ0gRRasH3zSuv397-lm_x1fX1FSGcV7E>
8+
* _Previous Minutes Google Doc: <https://docs.google.com/document/d/16l9D_OvAkeG6JoM_HRomHcFl--FTB61quvIWo9QfsVw>_
9+
10+
## Present
11+
12+
* Bradley Meck @bmeck (observer/GoDaddy/TC39)
13+
* Colin Ihrig @cjihrig (CTC)
14+
* Evan Lucas @evanlucas (CTC)
15+
* James M Snell @jasnell (CTC)
16+
* Josh Gavant @joshgav (observer/Microsoft)
17+
* Michael Dawson @mhdawson (CTC)
18+
* Julien Gilli @misterdjules (CTC)
19+
* Mikeal Rogers @mikeal (observer/Node.js Foundation)
20+
* Brian White @mscdex (CTC)
21+
* Ali Ijaz Sheikh @ofrobots (CTC)
22+
* Jenn Turner @renrutnnej (observer/Node.js Foundation)
23+
* Steven R. Loomis @srl295 (observer)
24+
* Sakthipriyan Vairamani @thefourtheye (observer)
25+
* Trevor Norris @trevnorris (CTC)
26+
* Rich Trott @Trott (CTC)
27+
28+
## Standup
29+
30+
* Bradley Meck @bmeck (observer/GoDaddy/TC39)
31+
* Vacation, work on import changes for ES spec
32+
* Colin Ihrig @cjihrig (CTC)
33+
* Reviewed issues and PRs. Revisited an old PR.
34+
* Evan Lucas @evanlucas (CTC)
35+
* cut v6.8.1 release
36+
* opened small doc PR
37+
* issue/pr review
38+
* James M Snell @jasnell (CTC)
39+
* Preparing v7.0.0 release
40+
* HTTP/2 implementation
41+
* PRs
42+
* Josh Gavant @joshgav (observer/Microsoft)
43+
* PR’s to improve user experience with new debugger
44+
* investigating how to integrate guides with API docs
45+
* off this week
46+
* Michael Dawson @mhdawson (CTC)
47+
* couple days vacation
48+
* Added code coverage nightly job and PR for doc
49+
* misc issue review/comment/lands
50+
* Some Abi stable node discussion/work
51+
* A number of benchmarking issues to look at
52+
* Julien Gilli @misterdjules (CTC)
53+
* nothing too significant
54+
* Mikeal Rogers @mikeal (observer/Node.js Foundation)
55+
* Putting together bugeting stuff
56+
* Brian White @mscdex (CTC)
57+
* Continued work on improving string encoding/decoding
58+
performance.
59+
* Commented on issues, reviewed PRs.
60+
* Ali Ijaz Sheikh @ofrobots (CTC)
61+
* Some V8 backport triage. Not much else notable.
62+
* Jenn Turner @renrutnnej (observer/Node.js Foundation)
63+
* Just observing, no update
64+
* Steven R. Loomis @srl295 (observer)
65+
* ICU 58 going GA end of week so will submit PR to update - [#7844](https://github.com/nodejs/node/issues/7844)
66+
* Backported break iterator fix to v4.x [#9008](https://github.com/nodejs/node/pull/9008)
67+
* Sakthipriyan Vairamani @thefourtheye (observer)
68+
* mostly PR reviews
69+
* Trevor Norris @trevnorris (CTC)
70+
* Worked with Matt Loring, proposed promise hooks api is
71+
sufficient combined with debugger API.
72+
* Rich Trott @Trott (CTC)
73+
* Outreachy
74+
* flaky tests
75+
* linting tools and rules
76+
77+
---
78+
79+
## Agenda
80+
81+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
82+
83+
### nodejs/node
84+
85+
* doc: add ctc-review label information [#9072](https://github.com/nodejs/node/pull/9072) @Trott
86+
* http: improve invalid character in header error message [#9010](https://github.com/nodejs/node/pull/9010) @evanlucas
87+
* net: multiple listen() events fail silently [#8419](https://github.com/nodejs/node/pull/8419) @jasnell
88+
89+
### nodejs/TSC
90+
91+
* Consider folding TSC into CTC [#146](https://github.com/nodejs/TSC/issues/146) @rvagg
92+
93+
94+
---
95+
96+
## Previous Meeting Review
97+
98+
### nodejs/node
99+
100+
* governance: expand use of CTC issue tracker [#8945](https://github.com/nodejs/node/pull/8945)
101+
102+
* Finalize through GitHub discussions.
103+
* Consider `ctc-review` label (#9072).
104+
105+
* doc: add supported platforms list [#8922](https://github.com/nodejs/node/pull/8922)
106+
107+
* Complete, remove from agenda.
108+
109+
* net: multiple listen() events fail silently [#8419](https://github.com/nodejs/node/pull/8419)
110+
111+
* Back to GitHub for further discussion.
112+
113+
### nodejs/TSC
114+
115+
* Consider folding TSC into CTC [#146](https://github.com/nodejs/TSC/issues/146)
116+
117+
---
118+
119+
## Minutes
120+
121+
### doc: add ctc-review label information [#9072](https://github.com/nodejs/node/pull/9072) @Trott
122+
123+
@trott: Has landed. If any objections speak now. Goal is more resolutions within tracker, less in meetings.
124+
125+
---
126+
127+
### http: improve invalid character in header error message [#9010](https://github.com/nodejs/node/pull/9010) @evanlucas
128+
129+
@evanlucas: Concerns about this leading to an information leak, because header name is displayed.
130+
131+
Would like to land this for v7. Would have to be merged today.
132+
133+
@trott: Question is if people are uncomfortable moving forward now before information from @Chalker is available.
134+
135+
@mhdawson: Seems there are already some hits (problems).
136+
137+
@evanlucas: Package `http-node` had problems, seems they copied `_http_outgoing.js`.
138+
139+
@evanlucas: Perhaps we can just add a debug message, that would not be semver-major.
140+
141+
@mhdawson: Seems better to do that than introduce a semver-major change this late.
142+
143+
@evanlucas: That seems fine, as long as we can get the needed info.
144+
145+
@trott: So land the debug message now, perhaps land standard output in v8.x.
146+
147+
**Next steps**:
148+
149+
* Create new PR with debug-based message, defer including in stderr/stdout till v8.x.
150+
151+
---
152+
153+
### net: multiple listen() events fail silently [#8419](https://github.com/nodejs/node/pull/8419) @jasnell
154+
155+
@jasnell: Question is whether to land in v7.x. If so please take it up today!
156+
157+
---
158+
159+
### Consider folding TSC into CTC [#146](https://github.com/nodejs/TSC/issues/146) @rvagg
160+
161+
@trott: Mainly to bring this to people’s attention, continue conversation in tracker.
162+
163+
---
164+
165+
### Info text in `--inspect` output [#8978](https://github.com/nodejs/node/pull/8978)
166+
167+
@joshgav: Should we land this for v7.x?
168+
169+
@trott: Is this a semver-major change?
170+
171+
@jasnell: No, especially cause this is an experimental feature.
172+
173+
@evanlucas: I’d prefer to wait till Chrome automatically lists Node.js targets via chrome://inspect, which is 55.
174+
175+
@ofrobots: No urgency to land this change now and it will make it harder to get started.
176+
177+
@mhdawson: Agreed, no urgency to land this before v7.x.
178+
179+
@joshgav: So I’ll update PR to include generic URL but not remove the chrome-devtools URL for now.
180+
181+
**Next steps**:
182+
183+
* Josh to update PR to still include chrome-devtools URL.
184+
* Consider removing chrome-devtools URL when 55 is released (early December).
185+
186+
---
187+
188+
## Q/A on public channels
189+
190+
191+
---
192+
193+
## Upcoming Meetings
194+
195+
* CTC: 2016-10-26, 9am Pacific
196+
* TSC: 2016-10-20, 1pm Pacific
197+
* Build: 2016-10-24 (?)
198+
* Diagnostics: first week of November
199+
* Benchmarking:
200+
* LTS:
201+
* Post-Mortem:
202+
* API:

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.