Skip to content

Commit d1a0a14

Browse files
mhdawsonMyles Borins
authored and
Myles Borins
committed
doc: Add CTC meeting minutes for 2016-05-04
PR-URL: #6579 Reviewed-By: Myles Borins <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: thefourtheye - Sakthipriyan Vairamani <[email protected]>
1 parent 08cd81b commit d1a0a14

File tree

1 file changed

+192
-0
lines changed

1 file changed

+192
-0
lines changed

doc/ctc-meetings/2016-05-04.md

+192
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,192 @@
1+
# Node Foundation CTC Meeting 2016-05-04
2+
3+
## Links
4+
5+
* **Audio Recording**: https://www.youtube.com/watch?v=3M95wsWs7qQ
6+
* **GitHub Issue**: https://github.com/nodejs/node/issues/6567
7+
* **Minutes Google Doc**: <https://docs.google.com/document/d/1eP0HyB452ZmoMlljCXJNyWO-BZyhzo9j6kNn17sDCEg>
8+
* _Previous Minutes Google Doc: <https://docs.google.com/document/d/16t1um6sXBlFZT2aCLqnlwumcEcM2K-_xeEFnnsA_rBQ>_
9+
10+
## Present
11+
12+
* Bradley Farias @bmeck (observer/modules EPS/GoDaddy)
13+
* Brian Terlson @bterlson (observer/modules EPS/tc39/Microsoft)
14+
* Сковорода Никита Андреевич @ChALkeR (CTC)
15+
* Colin Ihrig @cjihrig (CTC)
16+
* Evan Lucas @evanlucas (CTC)
17+
* Jeremiah Senkpiel @Fishrock123 (CTC)
18+
* James M Snell @jasnell (CTC)
19+
* Josh Gavant (observer/Microsoft)
20+
* Michael Dawson @mhdawson (CTC)
21+
* Brian White @mscdex (CTC)
22+
* Rod Vagg @rvagg (CTC)
23+
* Seth Thompson (observer/Google)
24+
* Shigeki Ohtsu @shigeki (CTC)
25+
* Steven R Loomis @srl295 (observer/IBM/ICU)
26+
* Trevor Norris @trevnorris (CTC)
27+
* Rich Trott @Trott (CTC)
28+
29+
30+
## Agenda
31+
32+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
33+
34+
### nodejs/node
35+
36+
* Revert 5950 [#6537](https://github.com/nodejs/node/pull/6537)
37+
* src: refactor constants, deprecate require('constants') [#6534](https://github.com/nodejs/node/pull/6534)
38+
* doc: refactor the changelog by version [#6503](https://github.com/nodejs/node/pull/6503)
39+
* Request for discussion of https://github.com/dherman/defense-of-dot-js vs EPS on modules
40+
41+
42+
43+
## Standup
44+
45+
* Сковорода Никита Андреевич:
46+
* Nothing too significant, some comments (as usual).
47+
48+
* Colin Ihrig:
49+
* A few PRs
50+
* Reviewing issues and PRs
51+
52+
* Evan Lucas:
53+
* Submitted a doc and util pr.
54+
* Working on a http bug.
55+
* Been trying to help some packages get updated to work with v6.
56+
* Left some comments on a few issues.
57+
* Have been working on a commit validator tool.
58+
59+
* Jeremiah Senkpiel:
60+
* v6 Breaking changes doc
61+
* working on v6.1.0 release
62+
* assorted PRs, Issues, EPs -- done a bunch of reviews
63+
64+
* James M Snell:
65+
* v6 Release
66+
* Fixed 2 util.inspect bugs (proxy and array length)
67+
* Helped troubleshoot buffer indexOf/lastIndexOf bug
68+
* Troubleshooting 5950 issues
69+
* Work on error code refactoring
70+
* Work on constants refactoring
71+
* Work on changelog refactoring
72+
* Ongoing http implementation security / spec compliance review
73+
* Working on compliance follow ups from vm summit
74+
* Investigating possible alternative solutions for stdout/stderr
75+
non-blocking issues
76+
77+
* Josh Gavant:
78+
* Testing use of trace-event macros, designing tracing module, reviewing AsyncWrap and alternatives.
79+
80+
* Michael Dawson:
81+
* Investigated ppc be release machine issues
82+
* validating be build from nightly release job
83+
* AsyncWrap EP review
84+
* switched benchmarking vof v6 over to new v6 branch
85+
* wrote/submitted stable ABI module EP
86+
* misc reviews/lands
87+
88+
* Brian White:
89+
* Worked on http server code refactor and other http performance improvements
90+
* Reviewed PRs/issues
91+
92+
* Rod Vagg:
93+
* Time off, now catching up
94+
95+
* Seth Thompson:
96+
* continue to work on getting v8 inspector to work with node
97+
* meeting with Ali as he.s visiting in person
98+
99+
* Shigeki Ohtsu:
100+
* Upgrading openssl and reviewing one PR for GCM IV
101+
102+
* Steven R Loomis:
103+
* turn/rebase/clean ci for https://github.com/nodejs/node/pull/6088 ( checkin ICU into master ) and https://github.com/nodejs/node/pull/4253 ( v8BreakIterator to throw error rather than fatal crash
104+
* opened Doodle poll to get Intl WG going again (monthly) https://github.com/nodejs/Intl/issues/33
105+
106+
* Trevor Norris:
107+
* Don't abort on access of invalid pointer returned by Unwrap PR
108+
* Tuning the AsyncWrap EP and writing patches as result
109+
110+
* Rich Trott:
111+
* Added undocumented-for-now -F flag to jslint.js to automatically fix lint issues. Dog-fooding it right now. Give it a shot if you.re brave.
112+
* Open PR to have `known_issues` tests run via CI and `make test`. PTAL. https://github.com/nodejs/node/pull/6559
113+
* Assessments for long-dormant issues.
114+
115+
* Bradley Farias:
116+
* Investigated counter proposal details to node EP for modules
117+
* Started test framework for modules EP
118+
119+
120+
## Review of last meeting
121+
122+
123+
* governance: add new collaborators XII [#6282](https://github.com/nodejs/node/issues/6282)
124+
* doc: doc-only deprecation for util.log() [#6161](https://github.com/nodejs/node/pull/6161)
125+
* Check (small) ICU into repo [#6088](https://github.com/nodejs/node/pull/6088)
126+
* doc, tls: deprecate createSecurePair [#6063](https://github.com/nodejs/node/pull/6063)
127+
* Planning for v6 [#5766](https://github.com/nodejs/node/issues/5766)
128+
* Introduce staging branch for stable release streams [#6306](https://github.com/nodejs/node/issues/6306)
129+
* ES6 Modules detection update / https://github.com/nodejs/node-eps/issues/13
130+
131+
132+
## Minutes
133+
134+
### Revert 5950 [#6537](https://github.com/nodejs/node/pull/6537)
135+
* James: Bug picked up in v6 for symlinked peer dependencies, our tests/CI didn.t pick it up unfortunately. Proposal is to revert the change and review it after adding some tests for this.
136+
* Rod: there was some discussion about a revert being semver major
137+
* Jeremiah: not optimal, reverting is technically a semver-major
138+
* Rod: does anyone have an objection to a revert at this stage?
139+
* Jeremiah: I believe some of the new behaviour was useful, is there a path to reintroducing it?
140+
* James: least invasive route would be to add the new behaviour behind a flag, there are some other approaches being investigated. The folks who proposed this change in the first place are behind the reversion at this stage.
141+
142+
### Request for discussion of https://github.com/dherman/defense-of-dot-js vs EPS on modules
143+
144+
145+
* Bradley: we merged our proposal last week and afterward a counter-proposal (that we.ve been waiting on for many weeks) showed up. The proposal is a package.json-based solution. It does address all of the major use-cases of Node and it does this in slightly different ways than we.ve seen before. It.s still lacking some existing behaviour of CJS files (you can.t detect some filepaths for CJS), e.g. ~/.config.js. It introduces a path-expansion mechanism. You add a module.root to your package.json. I still have concerns about this but it.s much better than the previous ones based on package.json. I.ll be having a more in-depth conversation with Brian Terlson (TC39/Microsoft), if anyone wants to join.
146+
* Jeremiah: is the main downside of the file extension about sharing files from servers that aren.t configured for it?
147+
* Bradley: yes, the main concern is about firewalls and configs that block transmission of the new files.
148+
* Rod: can you help us empathise with the sentimentality around the file extension, are there objective problems that we are not picking up on?
149+
* Bradley: there.s a concern . _something about a lodash example_ . shell scripts not finding the new files with *.js.
150+
* Brian Terlson: there.s an expectation that JavaScript is contained in a .js file and we.re in a nice place now that this is true with some minor exceptions. For TypeScript we.d have to add a .mts and .mtsx extension.
151+
* Rod: _asking about the sentimentality again_
152+
* Brian Terlson: that.s a mischaracterisation of the defense of .js proposal
153+
* Jeremiah: there will be developer friction either way
154+
* Brian Terlson: but friction will be limited to node developers as opposed to all javascript developers
155+
* Rod: what state is standard ?
156+
* Bradley: Some pieces still have minor changes going on but mostly locked down.
157+
* Brian Terlson: we have been thinking about all of these kinds of things during the whole process. I don.t think spec changes are off the table, TC39 could reconsider some aspects of the spec to improve matters. No promises, but don.t feel like you shouldn.t bring it up because we.re so late in process.
158+
* Bradley: forward detection is the main problem. Have examples of specifics. As long as implicit strict mode is enforced we are going to have a hard time.
159+
Rod: what.s the path forward ? On our side seems like strong preference is still on the file extension side.
160+
* Bradley: Either way was ecosystem effects. Personally towards file extensions, most response on social media is either .better than nothing. or .don.t like file extension but not pushing for package.json either.. Will discuss further with Brian and others from Microsoft.
161+
* Rod: EP does not mean it is final, but it does mean it is the preferred way forward and CTC members would have to be convinced otherwise.
162+
* Bradley: likely need longer discussion in next week or so either way.
163+
164+
165+
### src: refactor constants, deprecate require('constants') [#6534](https://github.com/nodejs/node/pull/6534)
166+
* James: recent PR asking to add new constant. They are mentioned in the docs in a few places, but not well documented. Figured should take a run at documenting and refactoring to make it make more sense. Cleans up the structure and properly documents. Would it be semver major since we have never documented before ?
167+
* Jeremiah: marked as semver major because it was deprecating existing constants.
168+
* James: hard deprecation pretty much blows up everything as npm uses them ,etc. Thats why it is a soft deprecation.
169+
* Jerimiah: we should find way to document
170+
* James: add to release notes saying to use the new versions.
171+
* Rich: Move discussion back to github
172+
* Rod: Any objection to doing this, want consensus before we spend a lot of time doing it.
173+
174+
175+
### doc: refactor the changelog by version [#6503](https://github.com/nodejs/node/pull/6503)
176+
177+
* James: changelog grew to point where it is no longer visible/usable on github. To make it at least visible it was temporarily split out into an archive. This PR is to split it out by version instead. Each version release would link to the individual version change log. One at top level becomes an index to the version specific ones. Acceptable or do we need another approach? Archive for io.js release and before v.10. Should not add too much additional work to release process.
178+
* Rod: maybe io.js ones should be in separate files as well instead archive.
179+
* Jeremiah: io.js v1 has particularly large set of changes
180+
* James: any major objections ?
181+
* Rod: take it back to github, roll forward unless objections on github.
182+
183+
### Q/A on public fora
184+
* call for questions on public channels.
185+
* no existing ones so far.
186+
* will wait a few minutes to see if any come in
187+
* ok moving on.
188+
189+
190+
## Next Meeting
191+
192+
2016-05-11

0 commit comments

Comments
 (0)