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 9dad98b

Browse filesBrowse files
TrottMyles Borins
authored andcommitted
doc: add 2016-10-26 CTC meeting minutes
PR-URL: #9348 Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 8240092 commit 9dad98b
Copy full SHA for 9dad98b

File tree

Expand file treeCollapse file tree

1 file changed

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

1 file changed

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

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

Copy file name to clipboard
+151Lines changed: 151 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,151 @@
1+
# Node Foundation CTC Meeting 2016-10-26
2+
3+
## Links
4+
5+
* **Audio Recording**: TBP
6+
* **GitHub Issue**:
7+
[#9261](https://github.com/nodejs/node/issues/9261)
8+
* **Minutes Google Doc**: <https://docs.google.com/document/d/1dUPFNyK8Yn1yWRAcXPY-mWM2KMWvOOB8Z5BH2s7RpJs>
9+
* _Previous Minutes Google Doc: <https://docs.google.com/document/d/18rnF8IfO10aQ0gRRasH3zSuv397-lm_x1fX1FSGcV7E>_
10+
11+
12+
## Present
13+
14+
* Anna Henningsen @addaleax (CTC)
15+
* Bradley Meck @bmeck (observer/GoDaddy/TC39)
16+
* Ben Noordhuis @bnoordhuis (CTC)
17+
* Сковорода Никита Андреевич @ChALkeR (CTC)
18+
* Colin Ihrig @cjihrig (CTC)
19+
* Evan Lucas @evanlucas (CTC)
20+
* James M Snell @jasnell (CTC)
21+
* Brian White @mscdex (CTC)
22+
* Ali Ijaz Sheikh @ofrobots (CTC)
23+
* Seth Thompson @s3ththompson (observer/Google)
24+
* Shigeki Ohtsu @shigeki (CTC)
25+
* Sakthipriyan Vairamani @thefourtheye (observer)
26+
* Rich Trott @Trott (CTC)
27+
28+
29+
## Standup
30+
31+
* Anna Henningsen @addaleax (CTC)
32+
* Not much
33+
* Bradley Meck @bmeck (observer/GoDaddy/TC39)
34+
* Work on making inspector work w/ vm
35+
* Minor talks w/ modules spec authors
36+
* Ben Noordhuis @bnoordhuis (CTC)
37+
* (Nothing reported.)
38+
* Сковорода Никита Андреевич @ChALkeR (CTC)
39+
* Rebuilt a new dataset from npm packages, some further work on the tooling
40+
* Some issue/pr comments as usual
41+
* Some ecosystem security stuff
42+
* Colin Ihrig @cjihrig (CTC)
43+
* Reviewing issues and PRs, opened a few PRs, libuv 1.10.0 update
44+
* Evan Lucas @evanlucas (CTC)
45+
* Working on getting libuv to use fsevents for file watching on OS X
46+
* Opened small PR to fix a test that kept failing on freebsd
47+
* James M Snell @jasnell (CTC)
48+
* Getting v7.0.0 out the door
49+
* PRs
50+
* More work on HTTP/2
51+
* Brian White @mscdex (CTC)
52+
* Continued working on string encoding/decoding performance. Starting to see even more promising results.
53+
* Reviewed PRs, commented on issues
54+
* Ali Ijaz Sheikh @ofrobots (CTC)
55+
* Not much
56+
* Seth Thompson @s3ththompson (observer/Google)
57+
* V8 5.5 beta shipped with async/await
58+
* Shigeki Ohtsu @shigeki (CTC)
59+
* Reviewed a few PR and made a security assessments of CVE-2016-8610 for Node-v0.10 and 0.12.
60+
* Sakthipriyan Vairamani @thefourtheye (observer)
61+
* Held an event to help people to get their first contribution into Node.js
62+
* Looking at V8 code base
63+
* Rich Trott @Trott (CTC)
64+
* Outreachy, Node Todo, Node Interactive prep
65+
* test and tools PRs
66+
* usual PR review/commenting
67+
68+
69+
---
70+
71+
72+
## Agenda
73+
74+
75+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
76+
77+
78+
### nodejs/TSC
79+
80+
81+
* Consider folding TSC into CTC [#146](https://github.com/nodejs/TSC/issues/146)
82+
83+
84+
### nodejs/node
85+
86+
87+
* Debugging: name every function
88+
[#8913]https://github.com/nodejs/node/issues/8913
89+
90+
91+
---
92+
93+
94+
## Previous Meeting Review
95+
96+
97+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
98+
99+
100+
### nodejs/node
101+
102+
103+
* doc: add ctc-review label information [#9072](https://github.com/nodejs/node/pull/9072) @Trott
104+
* http: improve invalid character in header error message [#9010](https://github.com/nodejs/node/pull/9010) @evanlucas
105+
* net: multiple listen() events fail silently [#8419](https://github.com/nodejs/node/pull/8419) @jasnell
106+
107+
108+
### nodejs/TSC
109+
110+
111+
* Consider folding TSC into CTC [#146](https://github.com/nodejs/TSC/issues/146) @rvagg
112+
113+
114+
---
115+
116+
117+
## Minutes
118+
119+
120+
### Consider folding TSC into CTC
121+
122+
123+
Rich: Defer until Rod is here and have the conversation in the issue tracker until then?
124+
125+
126+
James: Makes sense. Also, TSC call is tomorrow so it can be discussed then.
127+
128+
129+
### Debugging: name every function
130+
131+
132+
Rich: It seemed like a good idea initially, but it is not clear if it is a good thing in all cases (some +, some -). Issue is marked as good first contribution which means lots of new contributors are coming in.
133+
Myles wanted a quick resolution, but it is not clear there is a quick resolution. Someone needs to sit down and come up with a list of cases where it would be beneficial
134+
135+
136+
Brian: Prototype functions?
137+
138+
139+
Rich: That is a case where it doesn't add value but doesn't hurt either. There are other cases where it does remove information.
140+
141+
142+
Sakthipriyan: One example was fs.readFileSync..
143+
144+
145+
Brian: V8 may also be inferring names from a variable in cases when you do `let a = () => {...}`. Another case to take into consideration.
146+
147+
148+
Rich: Move back to the issue tracker; remove 'good-first-contribution' label until we have documented what should/shouldn't be done. I can work with someone, or come up with documentation.
149+
150+
151+
Floor: General agreement? Yes, general agreement.

0 commit comments

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